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

feat(bigquery/storage/managedwriter): add project autodetection #4605

Merged
merged 6 commits into from Aug 16, 2021

Conversation

shollyman
Copy link
Contributor

Towards: #4366

@shollyman shollyman requested a review from a team August 12, 2021 22:48
@shollyman shollyman requested a review from a team as a code owner August 12, 2021 22:48
@shollyman shollyman requested a review from tswast August 12, 2021 22:48
@product-auto-label product-auto-label bot added the api: bigquery Issues related to the BigQuery API. label Aug 12, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 12, 2021
@codyoss
Copy link
Member

codyoss commented Aug 13, 2021

If this is in not a big rush I suggest we wait to land this until #4582 lands. Once that lands and is released I would also like to refactor all other veneers so that this logic is consistent.

@shollyman
Copy link
Contributor Author

If this is in not a big rush I suggest we wait to land this until #4582 lands. Once that lands and is released I would also like to refactor all other veneers so that this logic is consistent.

updated logic to use internal/detect mechanism, but tests will fail here until we have a release with the change.

@shollyman shollyman merged commit d8cc9be into googleapis:master Aug 16, 2021
@shollyman shollyman deleted the detect-project branch August 16, 2021 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: bigquery Issues related to the BigQuery API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants