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.

Sign up for GitHub

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

Jump to bottom

fix: augment universe_domain handling #1837

Merged
merged 15 commits into from
Mar 5, 2024
Merged

fix: augment universe_domain handling #1837

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
a1e3276
fix: augment universe_domain handling
shollyman Feb 29, 2024
fa65569
Merge branch 'main' into universe-validation
shollyman Feb 29, 2024
a0126e6
lint
shollyman Feb 29, 2024
0cef28b
skipif core too old
shollyman Feb 29, 2024
506bb66
deps
shollyman Feb 29, 2024
e7cb75c
add import
shollyman Feb 29, 2024
5564bfd
no-cover in test helper
shollyman Mar 4, 2024
2848606
Merge branch 'main' into universe-validation
shollyman Mar 4, 2024
5ff2b5f
lint
shollyman Mar 4, 2024
0350b99
ignore google.auth typing
shollyman Mar 4, 2024
b69f580
capitalization
shollyman Mar 4, 2024
bccdae2
change to raise in test code
shollyman Mar 4, 2024
f746e94
Merge branch 'main' into universe-validation
Linchin Mar 4, 2024
2f5b49e
reviewer feedback
shollyman Mar 4, 2024
c1b8011
var fix
shollyman Mar 5, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Prev Previous commit
var fix
  • Loading branch information
@shollyman
shollyman committed Mar 5, 2024
commit c1b8011fe90f42e829c1fc62f9acae6b1f38191f
2 changes: 1 addition & 1 deletion google/cloud/bigquery/_helpers.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ def _get_client_universe(
universe = _DEFAULT_UNIVERSE
if hasattr(client_options, "universe_domain"):
options_universe = getattr(client_options, "universe_domain")
shollyman marked this conversation as resolved.
Show resolved Hide resolved
if options_universe is not None and len(env_universe) > 0:
if options_universe is not None and len(options_universe) > 0:
universe = options_universe
else:
env_universe = os.getenv(_UNIVERSE_DOMAIN_ENV)
Expand Down