Skip to content
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

Add deferrable mode to the PubSubPullOperator #45835

Merged

Conversation

molcay
Copy link
Contributor

@molcay molcay commented Jan 21, 2025

Add deferrable mode to the PubSubPullOperator


^ 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 Jan 21, 2025
@molcay molcay changed the title Add deferrable mode to the PubSubPullOperator [WIP] Add deferrable mode to the PubSubPullOperator Jan 21, 2025
@molcay molcay force-pushed the feat/add-deferrable-mode-pubsubpulloperator branch from 55da067 to 0bf8dd1 Compare January 28, 2025 10:02
@molcay molcay changed the title [WIP] Add deferrable mode to the PubSubPullOperator Add deferrable mode to the PubSubPullOperator Jan 29, 2025
@molcay molcay force-pushed the feat/add-deferrable-mode-pubsubpulloperator branch from 0bf8dd1 to b25d2ea Compare January 29, 2025 10:20
@potiuk
Copy link
Member

potiuk commented Feb 1, 2025

We are about to migrate Google to the new provider structure, so you will have to rebase your change with the moved files and fix the problems there.

@potiuk
Copy link
Member

potiuk commented Feb 5, 2025

Migrated. You should rebase @molcay

@olegkachur-e olegkachur-e force-pushed the feat/add-deferrable-mode-pubsubpulloperator branch 3 times, most recently from fad21df to 838495c Compare February 7, 2025 14:38
@olegkachur-e olegkachur-e force-pushed the feat/add-deferrable-mode-pubsubpulloperator branch from 838495c to 1fe51c6 Compare February 7, 2025 14:47
@VladaZakharova
Copy link
Contributor

@potiuk
Rebased, can you please check again? :)

@potiuk potiuk merged commit 69c58e2 into apache:main Feb 12, 2025
61 checks passed
@olegkachur-e olegkachur-e deleted the feat/add-deferrable-mode-pubsubpulloperator branch February 12, 2025 22:45
ambika-garg pushed a commit to ambika-garg/airflow that referenced this pull request Feb 17, 2025
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.

3 participants