Skip to content

Conversation

@kaxil
Copy link
Member

@kaxil kaxil commented May 23, 2025

We rely on workloads in Airflow 3 as opposed to passing airflow task run command in Executor


^ 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 airflow-core/newsfragments.

@boring-cyborg boring-cyborg bot added area:Executors-core LocalExecutor & SequentialExecutor area:providers area:Scheduler including HA (high availability) scheduler provider:celery labels May 23, 2025
@kaxil kaxil force-pushed the remove-old-executor-path branch 2 times, most recently from 945b902 to ed2383e Compare May 23, 2025 21:31
@kaxil kaxil marked this pull request as ready for review May 23, 2025 21:33
Copy link
Contributor

@jscheffl jscheffl left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks for keeping the compatibility in the providers!

@kaxil kaxil force-pushed the remove-old-executor-path branch from ed2383e to 3d6f3fb Compare May 23, 2025 22:07
We rely on workloads in Airflow 3 as opposed to passing `airflow task run` command in Executor
@kaxil kaxil force-pushed the remove-old-executor-path branch from 3d6f3fb to 1c5a7a0 Compare May 23, 2025 22:45
@kaxil kaxil merged commit 13e79e8 into apache:main May 23, 2025
96 checks passed
@kaxil kaxil deleted the remove-old-executor-path branch May 23, 2025 23:37
Copy link
Contributor

@amoghrajesh amoghrajesh left a comment

Choose a reason for hiding this comment

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

Nice

dadonnelly316 pushed a commit to dadonnelly316/airflow that referenced this pull request May 26, 2025
We rely on workloads in Airflow 3 as opposed to passing `airflow task run` command in Executor
sanederchik pushed a commit to sanederchik/airflow that referenced this pull request Jun 7, 2025
We rely on workloads in Airflow 3 as opposed to passing `airflow task run` command in Executor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:Executors-core LocalExecutor & SequentialExecutor area:providers area:Scheduler including HA (high availability) scheduler provider:celery

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants