Skip to content

Support a BigQuery quota project in BqUtils#717

Merged
kmontemayor2-sc merged 3 commits into
mainfrom
kmonte/bq-utils-quota-project
Jul 24, 2026
Merged

Support a BigQuery quota project in BqUtils#717
kmontemayor2-sc merged 3 commits into
mainfrom
kmonte/bq-utils-quota-project

Conversation

@kmontemayor2-sc

Copy link
Copy Markdown
Collaborator

What

  • Adds an optional quota_project_id argument to BqUtils.
  • Supports GIGL_BIGQUERY_QUOTA_PROJECT as a process-level default, with an explicit argument taking precedence.
  • Preserves the existing BigQuery client construction path when no non-empty override is set.
  • Propagates a configured quota project into launched Python component environments.
  • Documents configuration and the required Service Usage permission.

Why

BigQuery quota and API-enablement checks can need a project distinct from both the active credential project and the project containing BigQuery resources. A BigQuery-scoped client option supports that deployment model without changing quota behavior for other Google Cloud clients.

Testing

  • bq_test.py: 11 tests passed.
  • Quota environment propagation tests: 2 tests passed.
  • Scoped ty checks passed for all changed Python files.
  • Ruff and Markdown formatting passed for all changed files.

@mkolodner-sc mkolodner-sc left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, thanks!

@kmontemayor2-sc
kmontemayor2-sc marked this pull request as ready for review July 23, 2026 21:31
@kmontemayor2-sc
kmontemayor2-sc added this pull request to the merge queue Jul 24, 2026
Merged via the queue into main with commit a8de119 Jul 24, 2026
7 checks passed
@kmontemayor2-sc
kmontemayor2-sc deleted the kmonte/bq-utils-quota-project branch July 24, 2026 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants