Repository navigation
feat(bigquery): add bigquery_call_system_procedure sample to call a b… - #14740
Conversation
|
Here is the summary of changes. You are about to add 1 region tag.
This comment is generated by snippet-bot.
|
There was a problem hiding this comment.
Code Review
This pull request adds a new Python sample and tests for calling the BQ.REFRESH_MATERIALIZED_VIEW system procedure in BigQuery. The feedback recommends updating the ignored_versions in noxfile_config.py to ensure only the oldest and newest supported Python versions (3.10 and 3.13) are tested, ignoring intermediate versions.
|
|
||
| TEST_CONFIG_OVERRIDE = { | ||
| # You can opt out from the test for specific Python versions. | ||
| "ignored_versions": ["2.7", "3.7", "3.8", "3.9", "3.10", "3.11"], |
There was a problem hiding this comment.
According to the repository guidelines, we should only test the oldest and newest supported Python releases in test configurations, ignoring intermediate versions. Since the oldest supported version is Python 3.10 (as specified in requirements-test.txt) and the newest is Python 3.13, we should test 3.10 and 3.13, while ignoring the intermediate versions 3.11 and 3.12.
| "ignored_versions": ["2.7", "3.7", "3.8", "3.9", "3.10", "3.11"], | |
| "ignored_versions": ["2.7", "3.7", "3.8", "3.9", "3.11", "3.12"], |
References
- Only test the oldest and newest supported Python releases in test configurations (such as noxfile_config.py), ignoring intermediate versions.
692d9d0 to
e64649a
Compare
…uilt-in system procedure Problem: The BigQuery documentation (https://cloud.google.com/bigquery/docs/procedures#call_a_system_procedure) needs a standalone, tested Python code sample demonstrating how to call a built-in system procedure (region tag `bigquery_call_system_procedure`). Solution: - Add `call_system_procedure(project_id: str, dataset_id: str, materialized_view_id: str) -> None` in `bigquery/stored_procedures/bigquery_call_system_procedure.py` (Invoke a built-in BigQuery system procedure such as BQ.REFRESH_MATERIALIZED_VIEW using CALL.) - Add unit tests (`bigquery/stored_procedures/bigquery_call_system_procedure_test.py`) covering both the success path and API error handling (`GoogleAPICallError`)
e64649a to
4762353
Compare
…uilt-in system procedure
Problem:
The BigQuery documentation (https://cloud.google.com/bigquery/docs/procedures#call_a_system_procedure) needs a standalone, tested Python code sample demonstrating how to call a built-in system procedure (region tag
bigquery_call_system_procedure).Solution:
call_system_procedure(project_id: str, dataset_id: str, materialized_view_id: str) -> Noneinbigquery/stored_procedures/bigquery_call_system_procedure.py(Invoke a built-in BigQuery system procedure such as BQ.REFRESH_MATERIALIZED_VIEW using CALL.)bigquery/stored_procedures/bigquery_call_system_procedure_test.py) covering both the success path and API error handling (GoogleAPICallError)Description
Fixes #
Checklist
Testing
Compliance & Style
Post-Approval Actions