Skip to content

Conversation

@tedromer
Copy link
Contributor

@tedromer tedromer commented May 21, 2020

Add --region to python examples where it was missing.

R: @ibzib

# run your pipeline on the Google Cloud Dataflow Service.
'--project=SET_YOUR_PROJECT_ID_HERE',
# CHANGE 4/5: Your Google Cloud Storage path is required for staging local
# CHANGE 4/6: (OPTIONAL) The Google Cloud Engine region is
Copy link
Contributor

Choose a reason for hiding this comment

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

See https://cloud.google.com/dataflow/docs/concepts/regional-endpoints

Suggested change
# CHANGE 4/6: (OPTIONAL) The Google Cloud Engine region is
# CHANGE 4/6: (OPTIONAL) The Google Compute Engine region (such as us-central1) is

--test-pipeline-options="
--runner=TestDataflowRunner
--project=...
--region=...
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
--region=...
--region=...

@aaltay
Copy link
Member

aaltay commented May 28, 2020

retest this please

@aaltay
Copy link
Member

aaltay commented May 28, 2020

LGTM. I will merge after tests pass. Thank you @tedromer

@aaltay
Copy link
Member

aaltay commented May 30, 2020

retest this please

1 similar comment
@aaltay
Copy link
Member

aaltay commented May 30, 2020

retest this please

@aaltay aaltay merged commit 804dd5e into apache:master Jun 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants