Skip to content

Commit 4abbf6e

Browse files
mugiokaYuan325
andauthored
fix(bigquery): wire maximumBytesBilled into prebuilt config (googleapis#3385)
## Description `maximumBytesBilled` was added to the BigQuery source config in googleapis#2724 (v1.2.0), but was not wired into the prebuilt config (`internal/prebuiltconfigs/tools/bigquery.yaml`). Users running with `--prebuilt bigquery` had no way to set a per-query scan limit via environment variable. This change adds `BIGQUERY_MAXIMUM_BYTES_BILLED` to the prebuilt config, following the same pattern as other optional fields (`BIGQUERY_IMPERSONATE_SERVICE_ACCOUNT`, `BIGQUERY_SCOPES`, etc.). Fixes googleapis#3384 🦕 --------- Co-authored-by: Yuan Teoh <45984206+Yuan325@users.noreply.github.com>
1 parent 78d011a commit 4abbf6e

2 files changed

Lines changed: 4 additions & 0 deletions

File tree

‎docs/en/integrations/bigquery/prebuilt-configs/bigquery.md‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,9 @@ description: "Details of the BigQuery prebuilt configuration."
1818
to impersonate when making BigQuery and Dataplex API calls. The
1919
authenticated principal must have `roles/iam.serviceAccountTokenCreator`
2020
on the target service account.
21+
* `BIGQUERY_MAXIMUM_BYTES_BILLED`: (Optional) Per-query bytes scanned cap
22+
(in bytes). Queries that exceed this limit fail before executing and
23+
cost nothing.
2124
* **Permissions:**
2225
* **BigQuery User** (`roles/bigquery.user`) to execute queries and view
2326
metadata.

‎internal/prebuiltconfigs/tools/bigquery.yaml‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ useClientOAuth: ${BIGQUERY_USE_CLIENT_OAUTH:false}
2121
scopes: ${BIGQUERY_SCOPES:}
2222
maxQueryResultRows: ${BIGQUERY_MAX_QUERY_RESULT_ROWS:50}
2323
impersonateServiceAccount: ${BIGQUERY_IMPERSONATE_SERVICE_ACCOUNT:}
24+
maximumBytesBilled: ${BIGQUERY_MAXIMUM_BYTES_BILLED:0}
2425
---
2526
kind: tool
2627
name: analyze_contribution

0 commit comments

Comments
 (0)