Skip to content

Conversation

@shahar1
Copy link
Contributor

@shahar1 shahar1 commented Sep 8, 2023

closes: #11721


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

@boring-cyborg boring-cyborg bot added area:providers provider:google Google (including GCP) related issues labels Sep 8, 2023
Copy link
Contributor

@eladkal eladkal left a comment

Choose a reason for hiding this comment

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

LGTM!

Nice one @shahar1

@eladkal eladkal changed the title Add expected_terminal_state param. to Dataflow operators Add expected_terminal_state parameter to Dataflow operators Sep 8, 2023
@shahar1 shahar1 force-pushed the dataflow-expected-terminal-state branch from 7d6c237 to 80666bc Compare September 8, 2023 15:15
@potiuk potiuk merged commit 050a47a into apache:main Sep 11, 2023
@shahar1 shahar1 deleted the dataflow-expected-terminal-state branch June 12, 2024 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers provider:google Google (including GCP) related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dataflow operators - add user possibility to define expected terminal state

3 participants