Skip to content

Replace usage of DummyOperator with EmptyOperator - #22974

Merged
eladkal merged 7 commits into
apache:mainfrom
eladkal:emptyop
Apr 13, 2022
Merged

Replace usage of DummyOperator with EmptyOperator#22974
eladkal merged 7 commits into
apache:mainfrom
eladkal:emptyop

Conversation

@eladkal

@eladkal eladkal commented Apr 13, 2022

Copy link
Copy Markdown
Contributor

This is a followup PR to #22832

  1. Example dags in Providers are handled differently because Providers should be compatible with Airflow 2.1
  2. In core / tests all changed to EmptyOperator

^ 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.

@eladkal eladkal added this to the Airflow 2.3.0 milestone Apr 13, 2022
@boring-cyborg boring-cyborg Bot added area:API Airflow's REST/HTTP API area:production-image Production image improvements and fixes area:providers area:serialization kind:documentation provider:microsoft-azure Azure-related issues labels Apr 13, 2022
Comment thread tests/dags/test_only_empty_tasks.py Outdated
Comment thread airflow/providers/dbt/cloud/example_dags/example_dbt_cloud.py Outdated
Comment thread airflow/providers/dbt/cloud/example_dags/example_dbt_cloud.py Outdated
Comment thread airflow/operators/dummy_operator.py Outdated
@github-actions

Copy link
Copy Markdown
Contributor

The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the risk. If they don't merge it quickly - please rebase it to the latest main at your convenience, or amend the last commit of the PR, and push it with --force-with-lease.

@github-actions github-actions Bot added the full tests needed We need to run full set of tests for this PR to merge label Apr 13, 2022
@eladkal
eladkal merged commit 49e336a into apache:main Apr 13, 2022
@eladkal
eladkal deleted the emptyop branch April 13, 2022 19:28
@ephraimbuddy ephraimbuddy added the type:misc/internal Changelog: Misc changes that should appear in change log label Apr 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:API Airflow's REST/HTTP API area:production-image Production image improvements and fixes area:providers area:serialization full tests needed We need to run full set of tests for this PR to merge kind:documentation provider:microsoft-azure Azure-related issues type:misc/internal Changelog: Misc changes that should appear in change log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants