Skip to content

Removed deprecated Chainable type from BaseOperator - #42776

Merged
potiuk merged 8 commits into
apache:mainfrom
dirrao:chainable_datatype_removal
Oct 15, 2024
Merged

Removed deprecated Chainable type from BaseOperator#42776
potiuk merged 8 commits into
apache:mainfrom
dirrao:chainable_datatype_removal

Conversation

@dirrao

Copy link
Copy Markdown
Contributor

Removed deprecated Chainable type from BaseOperator

@dirrao
dirrao requested a review from uranusjr as a code ownerOctober 6, 2024 04:16
@dirrao
dirrao requested review from eladkal and potiukOctober 6, 2024 04:36
@uranusjr

Copy link
Copy Markdown
Member

We should also remove this word from spelling_wordlist.txt.

Also, apparently we forgot to actually deprecate this, so we’ll need an additional PR to add the deprecation warning to the 2.x branch.

@dirrao

Copy link
Copy Markdown
ContributorAuthor

We should also remove this word from spelling_wordlist.txt.

Done

Also, apparently we forgot to actually deprecate this, so we’ll need an additional PR to add the deprecation warning to the 2.x branch.

Make sense. we will create new PR for the same.

@shahar1shahar1 added airflow3.0:breaking Candidates for Airflow 3.0 that contain breaking changes area:core-operators airflow3.0:candidate Potential candidates for Airflow 3.0 labels Oct 7, 2024

@shahar1shahar1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Breaking change, please add a newsfragment :)

@dirraodirrao self-assigned this Oct 8, 2024
@dirrao

Copy link
Copy Markdown
ContributorAuthor

Breaking change, please add a newsfragment :)

Done.

@dirrao
dirrao requested a review from shahar1October 10, 2024 10:16
@potiuk
potiuk merged commit 272cd9a into apache:mainOct 15, 2024
R7L208 pushed a commit to R7L208/airflow that referenced this pull request Oct 17, 2024
* Removed deprecated Chainable type from BaseOperator
* Chainable word removed from spelling_wordlist.txt
* Chainable word removed from spelling_wordlist.txt
* Chainable word removed from spelling_wordlist.txt
* news fragment added
harjeevanmaan pushed a commit to harjeevanmaan/airflow that referenced this pull request Oct 23, 2024
* Removed deprecated Chainable type from BaseOperator
* Chainable word removed from spelling_wordlist.txt
* Chainable word removed from spelling_wordlist.txt
* Chainable word removed from spelling_wordlist.txt
* news fragment added
PaulKobow7536 pushed a commit to PaulKobow7536/airflow that referenced this pull request Oct 24, 2024
* Removed deprecated Chainable type from BaseOperator
* Chainable word removed from spelling_wordlist.txt
* Chainable word removed from spelling_wordlist.txt
* Chainable word removed from spelling_wordlist.txt
* news fragment added
ellisms pushed a commit to ellisms/airflow that referenced this pull request Nov 13, 2024
* Removed deprecated Chainable type from BaseOperator
* Chainable word removed from spelling_wordlist.txt
* Chainable word removed from spelling_wordlist.txt
* Chainable word removed from spelling_wordlist.txt
* news fragment added
@Lee-WLee-W mentioned this pull request Nov 18, 2024
2 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

airflow3.0:breakingCandidates for Airflow 3.0 that contain breaking changesairflow3.0:candidatePotential candidates for Airflow 3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@dirrao@uranusjr@potiuk@shahar1