Skip to content

Conversation

@ashb
Copy link
Member

@ashb ashb commented Nov 3, 2020

We have deprecated this in #12069 (for inclusion in 1.10.13) and the
docs http://airflow.apache.org/docs/stable/howto/custom-operator.html
already show how to do this without a plugin.

Closes #9498


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, 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 UPDATING.md.

@ashb ashb added area:plugins full tests needed We need to run full set of tests for this PR to merge labels Nov 3, 2020
@ashb ashb requested a review from kaxil November 3, 2020 17:27
Copy link
Member

@XD-DENG XD-DENG left a comment

Choose a reason for hiding this comment

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

I may be wrong, but does

plugins_attributes_to_dump = [
need to be updated?

@ashb
Copy link
Member Author

ashb commented Nov 3, 2020

Yes good catch @XD-DENG - missed that one in my grep.

@ashb ashb force-pushed the remove-adding-operators-via-plugins branch from 54859d5 to b844b2d Compare November 3, 2020 19:58
@kaxil kaxil force-pushed the remove-adding-operators-via-plugins branch 2 times, most recently from 89f4b10 to 9bcecc2 Compare November 3, 2020 23:59
We have deprecated this in apache#12069 (for inclusion in 1.10.13) and the
docs http://airflow.apache.org/docs/stable/howto/custom-operator.html
already show how to do this without a plugin.

Closes apache#9498
@kaxil kaxil force-pushed the remove-adding-operators-via-plugins branch from 9bcecc2 to 0dcde7a Compare November 4, 2020 00:52
@kaxil kaxil merged commit 5e8b537 into apache:master Nov 4, 2020
@kaxil kaxil deleted the remove-adding-operators-via-plugins branch November 4, 2020 01:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:CLI area:plugins full tests needed We need to run full set of tests for this PR to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove adding Operators via plugins in 2.0

3 participants