-
Notifications
You must be signed in to change notification settings - Fork 717
feat(bigquery-execute-sql): add dry run support #1057
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
Conversation
Hi @Genesis929 , can you update the documentation as well? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! :)
+1 @averikitsch comment on adding documentation
internal/tools/bigquery/bigqueryexecutesql/bigqueryexecutesql.go
Outdated
Show resolved
Hide resolved
internal/tools/bigquery/bigqueryexecutesql/bigqueryexecutesql.go
Outdated
Show resolved
Hide resolved
internal/tools/bigquery/bigqueryexecutesql/bigqueryexecutesql.go
Outdated
Show resolved
Hide resolved
Co-authored-by: Yuan Teoh <45984206+Yuan325@users.noreply.github.com>
Co-authored-by: Yuan Teoh <45984206+Yuan325@users.noreply.github.com>
Co-authored-by: Yuan Teoh <45984206+Yuan325@users.noreply.github.com>
Thanks! Doc updated, forgot that. |
/gcbrun |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Genesis929 Thank you!
/gcbrun |
Add optional `dry_run` parameter to bigquery-execute-sql, which defaults to `false`. When the `dry_run` parameter is set to `true`, the tool returns the metadata from the dry run instead of executing the query. Fixes #703 --------- Co-authored-by: Yuan Teoh <45984206+Yuan325@users.noreply.github.com> 1cac9b5
) Add optional `dry_run` parameter to bigquery-execute-sql, which defaults to `false`. When the `dry_run` parameter is set to `true`, the tool returns the metadata from the dry run instead of executing the query. Fixes googleapis#703 --------- Co-authored-by: Yuan Teoh <45984206+Yuan325@users.noreply.github.com> 1cac9b5
) Add optional `dry_run` parameter to bigquery-execute-sql, which defaults to `false`. When the `dry_run` parameter is set to `true`, the tool returns the metadata from the dry run instead of executing the query. Fixes googleapis#703 --------- Co-authored-by: Yuan Teoh <45984206+Yuan325@users.noreply.github.com> 1cac9b5
) Add optional `dry_run` parameter to bigquery-execute-sql, which defaults to `false`. When the `dry_run` parameter is set to `true`, the tool returns the metadata from the dry run instead of executing the query. Fixes googleapis#703 --------- Co-authored-by: Yuan Teoh <45984206+Yuan325@users.noreply.github.com> 1cac9b5
) Add optional `dry_run` parameter to bigquery-execute-sql, which defaults to `false`. When the `dry_run` parameter is set to `true`, the tool returns the metadata from the dry run instead of executing the query. Fixes googleapis#703 --------- Co-authored-by: Yuan Teoh <45984206+Yuan325@users.noreply.github.com> 1cac9b5
) Add optional `dry_run` parameter to bigquery-execute-sql, which defaults to `false`. When the `dry_run` parameter is set to `true`, the tool returns the metadata from the dry run instead of executing the query. Fixes googleapis#703 --------- Co-authored-by: Yuan Teoh <45984206+Yuan325@users.noreply.github.com> 1cac9b5
Add optional `dry_run` parameter to bigquery-execute-sql, which defaults to `false`. When the `dry_run` parameter is set to `true`, the tool returns the metadata from the dry run instead of executing the query. Fixes googleapis#703 --------- Co-authored-by: Yuan Teoh <45984206+Yuan325@users.noreply.github.com>
🤖 I have created a release *beep* *boop* --- ## [0.12.0](v0.11.0...v0.12.0) (2025-08-14) ### Features * **prebuiltconfig:** Introduce additional parameter to limit context in list_tables ([#1151](#1151)) ([497d3b1](497d3b1)) * **prebuiltconfig/alloydb-admin:** Add list cluster, instance and users ([#1126](#1126)) ([b42c139](b42c139)) * **prebuiltconfig/alloydb-postgres:** Add tool to create user via Built in user type or IAM ([#1130](#1130)) ([f5bcb9c](f5bcb9c)) * **source/http:** Add User Agent to `http` invocations ([#1102](#1102)) ([6f55b78](6f55b78)) * **sources/postgres:** Add support for `queryParams` ([#1047](#1047)) ([7b57251](7b57251)), closes [#963](#963) * **tools/bigquery-execute-sql:** Add dry run support ([#1057](#1057)) ([1cac9b5](1cac9b5)) * **tools/dataplex-search-aspect-types:** Add support for `dataplex-search-aspect-types` tool ([#1061](#1061)) ([d940187](d940187)) * **tools/looker:** Add `looker-make-look` tool to create Looks ([#1099](#1099)) ([61d9489](61d9489)) * **tools/looker:** Add visualizations to `query-url` tool ([#1090](#1090)) ([5bf2758](5bf2758)) * **tools/looker:** New Looker tools for dashboards ([#1118](#1118)) ([42be3f5](42be3f5)) * **ui:** Add login with google button for automatic id token retrieval ([#1044](#1044)) ([d91bdfc](d91bdfc)) ### Bug Fixes * Correct the capitalization of `map` manifests ([#1139](#1139)) ([0b0457c](0b0457c)) * Remove unnecessary fields from `map` parameter manifests ([#1138](#1138)) ([fbe8c1a](fbe8c1a)) * **tools/looker:** Add authorized invocation feature to all Looker tools ([#1091](#1091)) ([3b1cce7](3b1cce7)) * Update ui info log to reflect port ([#1125](#1125)) ([6d691d5](6d691d5)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Yuan Teoh <45984206+Yuan325@users.noreply.github.com>
🤖 I have created a release *beep* *boop* --- ## [0.12.0](v0.11.0...v0.12.0) (2025-08-14) ### Features * **prebuiltconfig:** Introduce additional parameter to limit context in list_tables ([#1151](#1151)) ([497d3b1](497d3b1)) * **prebuiltconfig/alloydb-admin:** Add list cluster, instance and users ([#1126](#1126)) ([b42c139](b42c139)) * **prebuiltconfig/alloydb-postgres:** Add tool to create user via Built in user type or IAM ([#1130](#1130)) ([f5bcb9c](f5bcb9c)) * **source/http:** Add User Agent to `http` invocations ([#1102](#1102)) ([6f55b78](6f55b78)) * **sources/postgres:** Add support for `queryParams` ([#1047](#1047)) ([7b57251](7b57251)), closes [#963](#963) * **tools/bigquery-execute-sql:** Add dry run support ([#1057](#1057)) ([1cac9b5](1cac9b5)) * **tools/dataplex-search-aspect-types:** Add support for `dataplex-search-aspect-types` tool ([#1061](#1061)) ([d940187](d940187)) * **tools/looker:** Add `looker-make-look` tool to create Looks ([#1099](#1099)) ([61d9489](61d9489)) * **tools/looker:** Add visualizations to `query-url` tool ([#1090](#1090)) ([5bf2758](5bf2758)) * **tools/looker:** New Looker tools for dashboards ([#1118](#1118)) ([42be3f5](42be3f5)) * **ui:** Add login with google button for automatic id token retrieval ([#1044](#1044)) ([d91bdfc](d91bdfc)) ### Bug Fixes * Correct the capitalization of `map` manifests ([#1139](#1139)) ([0b0457c](0b0457c)) * Remove unnecessary fields from `map` parameter manifests ([#1138](#1138)) ([fbe8c1a](fbe8c1a)) * **tools/looker:** Add authorized invocation feature to all Looker tools ([#1091](#1091)) ([3b1cce7](3b1cce7)) * Update ui info log to reflect port ([#1125](#1125)) ([6d691d5](6d691d5)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Yuan Teoh <45984206+Yuan325@users.noreply.github.com> d19cfc1
🤖 I have created a release *beep* *boop* --- ## [0.12.0](googleapis/genai-toolbox@v0.11.0...v0.12.0) (2025-08-14) ### Features * **prebuiltconfig:** Introduce additional parameter to limit context in list_tables ([googleapis#1151](googleapis#1151)) ([497d3b1](googleapis@497d3b1)) * **prebuiltconfig/alloydb-admin:** Add list cluster, instance and users ([googleapis#1126](googleapis#1126)) ([b42c139](googleapis@b42c139)) * **prebuiltconfig/alloydb-postgres:** Add tool to create user via Built in user type or IAM ([googleapis#1130](googleapis#1130)) ([f5bcb9c](googleapis@f5bcb9c)) * **source/http:** Add User Agent to `http` invocations ([googleapis#1102](googleapis#1102)) ([6f55b78](googleapis@6f55b78)) * **sources/postgres:** Add support for `queryParams` ([googleapis#1047](googleapis#1047)) ([7b57251](googleapis@7b57251)), closes [googleapis#963](googleapis#963) * **tools/bigquery-execute-sql:** Add dry run support ([googleapis#1057](googleapis#1057)) ([1cac9b5](googleapis@1cac9b5)) * **tools/dataplex-search-aspect-types:** Add support for `dataplex-search-aspect-types` tool ([googleapis#1061](googleapis#1061)) ([d940187](googleapis@d940187)) * **tools/looker:** Add `looker-make-look` tool to create Looks ([googleapis#1099](googleapis#1099)) ([61d9489](googleapis@61d9489)) * **tools/looker:** Add visualizations to `query-url` tool ([googleapis#1090](googleapis#1090)) ([5bf2758](googleapis@5bf2758)) * **tools/looker:** New Looker tools for dashboards ([googleapis#1118](googleapis#1118)) ([42be3f5](googleapis@42be3f5)) * **ui:** Add login with google button for automatic id token retrieval ([googleapis#1044](googleapis#1044)) ([d91bdfc](googleapis@d91bdfc)) ### Bug Fixes * Correct the capitalization of `map` manifests ([googleapis#1139](googleapis#1139)) ([0b0457c](googleapis@0b0457c)) * Remove unnecessary fields from `map` parameter manifests ([googleapis#1138](googleapis#1138)) ([fbe8c1a](googleapis@fbe8c1a)) * **tools/looker:** Add authorized invocation feature to all Looker tools ([googleapis#1091](googleapis#1091)) ([3b1cce7](googleapis@3b1cce7)) * Update ui info log to reflect port ([googleapis#1125](googleapis#1125)) ([6d691d5](googleapis@6d691d5)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Yuan Teoh <45984206+Yuan325@users.noreply.github.com> d19cfc1
🤖 I have created a release *beep* *boop* --- ## [0.12.0](googleapis/genai-toolbox@v0.11.0...v0.12.0) (2025-08-14) ### Features * **prebuiltconfig:** Introduce additional parameter to limit context in list_tables ([googleapis#1151](googleapis#1151)) ([497d3b1](googleapis@497d3b1)) * **prebuiltconfig/alloydb-admin:** Add list cluster, instance and users ([googleapis#1126](googleapis#1126)) ([b42c139](googleapis@b42c139)) * **prebuiltconfig/alloydb-postgres:** Add tool to create user via Built in user type or IAM ([googleapis#1130](googleapis#1130)) ([f5bcb9c](googleapis@f5bcb9c)) * **source/http:** Add User Agent to `http` invocations ([googleapis#1102](googleapis#1102)) ([6f55b78](googleapis@6f55b78)) * **sources/postgres:** Add support for `queryParams` ([googleapis#1047](googleapis#1047)) ([7b57251](googleapis@7b57251)), closes [googleapis#963](googleapis#963) * **tools/bigquery-execute-sql:** Add dry run support ([googleapis#1057](googleapis#1057)) ([1cac9b5](googleapis@1cac9b5)) * **tools/dataplex-search-aspect-types:** Add support for `dataplex-search-aspect-types` tool ([googleapis#1061](googleapis#1061)) ([d940187](googleapis@d940187)) * **tools/looker:** Add `looker-make-look` tool to create Looks ([googleapis#1099](googleapis#1099)) ([61d9489](googleapis@61d9489)) * **tools/looker:** Add visualizations to `query-url` tool ([googleapis#1090](googleapis#1090)) ([5bf2758](googleapis@5bf2758)) * **tools/looker:** New Looker tools for dashboards ([googleapis#1118](googleapis#1118)) ([42be3f5](googleapis@42be3f5)) * **ui:** Add login with google button for automatic id token retrieval ([googleapis#1044](googleapis#1044)) ([d91bdfc](googleapis@d91bdfc)) ### Bug Fixes * Correct the capitalization of `map` manifests ([googleapis#1139](googleapis#1139)) ([0b0457c](googleapis@0b0457c)) * Remove unnecessary fields from `map` parameter manifests ([googleapis#1138](googleapis#1138)) ([fbe8c1a](googleapis@fbe8c1a)) * **tools/looker:** Add authorized invocation feature to all Looker tools ([googleapis#1091](googleapis#1091)) ([3b1cce7](googleapis@3b1cce7)) * Update ui info log to reflect port ([googleapis#1125](googleapis#1125)) ([6d691d5](googleapis@6d691d5)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Yuan Teoh <45984206+Yuan325@users.noreply.github.com> d19cfc1
🤖 I have created a release *beep* *boop* --- ## [0.12.0](googleapis/genai-toolbox@v0.11.0...v0.12.0) (2025-08-14) ### Features * **prebuiltconfig:** Introduce additional parameter to limit context in list_tables ([googleapis#1151](googleapis#1151)) ([497d3b1](googleapis@497d3b1)) * **prebuiltconfig/alloydb-admin:** Add list cluster, instance and users ([googleapis#1126](googleapis#1126)) ([b42c139](googleapis@b42c139)) * **prebuiltconfig/alloydb-postgres:** Add tool to create user via Built in user type or IAM ([googleapis#1130](googleapis#1130)) ([f5bcb9c](googleapis@f5bcb9c)) * **source/http:** Add User Agent to `http` invocations ([googleapis#1102](googleapis#1102)) ([6f55b78](googleapis@6f55b78)) * **sources/postgres:** Add support for `queryParams` ([googleapis#1047](googleapis#1047)) ([7b57251](googleapis@7b57251)), closes [googleapis#963](googleapis#963) * **tools/bigquery-execute-sql:** Add dry run support ([googleapis#1057](googleapis#1057)) ([1cac9b5](googleapis@1cac9b5)) * **tools/dataplex-search-aspect-types:** Add support for `dataplex-search-aspect-types` tool ([googleapis#1061](googleapis#1061)) ([d940187](googleapis@d940187)) * **tools/looker:** Add `looker-make-look` tool to create Looks ([googleapis#1099](googleapis#1099)) ([61d9489](googleapis@61d9489)) * **tools/looker:** Add visualizations to `query-url` tool ([googleapis#1090](googleapis#1090)) ([5bf2758](googleapis@5bf2758)) * **tools/looker:** New Looker tools for dashboards ([googleapis#1118](googleapis#1118)) ([42be3f5](googleapis@42be3f5)) * **ui:** Add login with google button for automatic id token retrieval ([googleapis#1044](googleapis#1044)) ([d91bdfc](googleapis@d91bdfc)) ### Bug Fixes * Correct the capitalization of `map` manifests ([googleapis#1139](googleapis#1139)) ([0b0457c](googleapis@0b0457c)) * Remove unnecessary fields from `map` parameter manifests ([googleapis#1138](googleapis#1138)) ([fbe8c1a](googleapis@fbe8c1a)) * **tools/looker:** Add authorized invocation feature to all Looker tools ([googleapis#1091](googleapis#1091)) ([3b1cce7](googleapis@3b1cce7)) * Update ui info log to reflect port ([googleapis#1125](googleapis#1125)) ([6d691d5](googleapis@6d691d5)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Yuan Teoh <45984206+Yuan325@users.noreply.github.com> d19cfc1
🤖 I have created a release *beep* *boop* --- ## [0.12.0](v0.11.0...v0.12.0) (2025-08-14) ### Features * **prebuiltconfig:** Introduce additional parameter to limit context in list_tables ([#1151](#1151)) ([497d3b1](497d3b1)) * **prebuiltconfig/alloydb-admin:** Add list cluster, instance and users ([#1126](#1126)) ([b42c139](b42c139)) * **prebuiltconfig/alloydb-postgres:** Add tool to create user via Built in user type or IAM ([#1130](#1130)) ([f5bcb9c](f5bcb9c)) * **source/http:** Add User Agent to `http` invocations ([#1102](#1102)) ([6f55b78](6f55b78)) * **sources/postgres:** Add support for `queryParams` ([#1047](#1047)) ([7b57251](7b57251)), closes [#963](#963) * **tools/bigquery-execute-sql:** Add dry run support ([#1057](#1057)) ([1cac9b5](1cac9b5)) * **tools/dataplex-search-aspect-types:** Add support for `dataplex-search-aspect-types` tool ([#1061](#1061)) ([d940187](d940187)) * **tools/looker:** Add `looker-make-look` tool to create Looks ([#1099](#1099)) ([61d9489](61d9489)) * **tools/looker:** Add visualizations to `query-url` tool ([#1090](#1090)) ([5bf2758](5bf2758)) * **tools/looker:** New Looker tools for dashboards ([#1118](#1118)) ([42be3f5](42be3f5)) * **ui:** Add login with google button for automatic id token retrieval ([#1044](#1044)) ([d91bdfc](d91bdfc)) ### Bug Fixes * Correct the capitalization of `map` manifests ([#1139](#1139)) ([0b0457c](0b0457c)) * Remove unnecessary fields from `map` parameter manifests ([#1138](#1138)) ([fbe8c1a](fbe8c1a)) * **tools/looker:** Add authorized invocation feature to all Looker tools ([#1091](#1091)) ([3b1cce7](3b1cce7)) * Update ui info log to reflect port ([#1125](#1125)) ([6d691d5](6d691d5)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Yuan Teoh <45984206+Yuan325@users.noreply.github.com>
Add optional
dry_run
parameter to bigquery-execute-sql, which defaults tofalse
.When the
dry_run
parameter is set totrue
, the tool returns the metadata from the dry run instead of executing the query.Fixes #703