GoogleCloudPlatform / GoogleCloudPlatform/workflows-demos

Include location parameter in the body example

Open
#134 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
159
Forks
43
PR merge metrics
No merged PRs in 30d

Description

https://github.com/GoogleCloudPlatform/workflows-demos/blob/4c4fbd91317f3a0293092d27b37eda1008403af0/bigquery-parallel/workflow-parallel.yaml#L44

Based on [QueryRequest Docs](https://cloud.google.com/workflows/docs/reference/googleapis/bigquery/v2/Overview#QueryRequest) and my tests, **location** atribute should be provided in the example for user convinience. It is not required, but it will be commonly required (e.g. customer using multi-regional datasets).

````yaml
body:
location: us #See details at https://cloud.google.com/bigquery/docs/locations#specifying_your_location
useLegacySql: false
useQueryCache: false
timeoutMs: 30000
# Find top 100 titles with most views on Wikipedia
query: ${
````

Contributor guide

Open the contributing guide

Research direction

Open bigquery-parallel/workflow-parallel.yaml at line 44 and compare the body with the linked QueryRequest documentation. Add the optional location example shown in the issue, then verify the YAML remains valid and the BigQuery workflow example still reflects the intended request body.

Written by the indexing model from the issue text.

Assessment

Tech stack
google-cloud
Domain
cloud
Issue type
Documentation
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.