Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PLUGIN-1727] Remove opencensus-impl dependency from GCS connector #1342

Conversation

arjan-bal
Copy link
Contributor

@arjan-bal arjan-bal commented Dec 2, 2023

PLUGIN-1727

This PR removes the only dependency on opencensus-impl and adds a build rule to ensure build fails when if the dependency is re-introduced again. See Jira for more details.

@arjan-bal arjan-bal force-pushed the PLUGIN-1727/remove-open-census-impl branch 2 times, most recently from 87117a9 to e51d7e5 Compare December 2, 2023 19:29
@itsankit-google itsankit-google added the build Trigger unit test build label Dec 4, 2023
@arjan-bal arjan-bal requested review from tivv and fernst December 4, 2023 05:40
@arjan-bal arjan-bal self-assigned this Dec 4, 2023
@arjan-bal arjan-bal force-pushed the PLUGIN-1727/remove-open-census-impl branch from c0e9351 to e51d7e5 Compare December 5, 2023 15:56
@arjan-bal arjan-bal force-pushed the PLUGIN-1727/remove-open-census-impl branch from e51d7e5 to 6cca7b6 Compare December 5, 2023 15:57
@arjan-bal arjan-bal merged commit 8e43f42 into data-integrations:develop Dec 5, 2023
13 checks passed
@arjan-bal arjan-bal deleted the PLUGIN-1727/remove-open-census-impl branch December 5, 2023 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Trigger unit test build
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants