Skip to content
This repository was archived by the owner on Aug 31, 2022. It is now read-only.

bq: use SA project id as default - #131

Merged
danielnorberg merged 4 commits into
masterfrom
fix-bq-default-project-id
Aug 8, 2018
Merged

danielnorberg merged 4 commits into
masterfrom
fix-bq-default-project-id

Conversation

@danielnorberg

Copy link
Copy Markdown
Contributor

Work around a bug google-cloud-java bug introduced in googleapis/google-cloud-java#2304

Also make BigQueryClientSingleton package-private.

Work around a bug google-cloud-java bug introduced in 
googleapis/google-cloud-java#2304

Also make BigQueryClientSingleton package-private.
@danielnorberg

Copy link
Copy Markdown
Contributor Author

Also opened an issue on google-cloud-java.

@honnix

honnix commented Aug 7, 2018

Copy link
Copy Markdown
Member

LGTM

// https://github.com/GoogleCloudPlatform/google-cloud-java/issues/3533
final String serviceAccountProjectId = getServiceAccountProjectId();
if (serviceAccountProjectId != null) {
bigquery.setProjectId(serviceAccountProjectId);

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This would override the GOOGLE_CLOUD_PROJECT env var if set by the user =/

@NatashaL

NatashaL commented Aug 8, 2018

Copy link
Copy Markdown
Contributor

👍

@ulzha

ulzha commented Aug 8, 2018

Copy link
Copy Markdown
Contributor

🚢

@honnix

honnix commented Aug 8, 2018

Copy link
Copy Markdown
Member

🤩

@danielnorberg
danielnorberg merged commit 61d499e into master Aug 8, 2018
@danielnorberg
danielnorberg deleted the fix-bq-default-project-id branch August 8, 2018 09:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants