Skip to content

Support BaseExecutor.run_workload() for Airflow 3.3 in Edge Worker#65847

Merged
potiuk merged 2 commits intoapache:mainfrom
jscheffl:feature/support-base-executor-run-workload-in-edge-after-62645
May 10, 2026
Merged

Support BaseExecutor.run_workload() for Airflow 3.3 in Edge Worker#65847
potiuk merged 2 commits intoapache:mainfrom
jscheffl:feature/support-base-executor-run-workload-in-edge-after-62645

Conversation

@jscheffl
Copy link
Copy Markdown
Contributor

While testing EdgeWorker I saw the message supervise() is deprecated, use supervise_task() instead. which seems to result from #62645 being merged...

So this PR adjusts the use of calling Supervisor from EdgeWorker in order to remove the deprecation. If Airflow >=3.3.0 it uses the BaseExecutor way.


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

@jscheffl jscheffl requested a review from dheerajturaga as a code owner April 25, 2026 18:15
@boring-cyborg boring-cyborg Bot added area:providers provider:edge Edge Executor / Worker (AIP-69) / edge3 labels Apr 25, 2026
@jscheffl jscheffl force-pushed the feature/support-base-executor-run-workload-in-edge-after-62645 branch from 5288365 to 3f81688 Compare April 29, 2026 05:21
@potiuk
Copy link
Copy Markdown
Member

potiuk commented May 10, 2026

conflicts :) ?

@jscheffl jscheffl force-pushed the feature/support-base-executor-run-workload-in-edge-after-62645 branch from 3f81688 to 148f85d Compare May 10, 2026 14:07
@jscheffl
Copy link
Copy Markdown
Contributor Author

conflicts :) ?

Fixed.

@potiuk potiuk merged commit 409895a into apache:main May 10, 2026
169 of 170 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers provider:edge Edge Executor / Worker (AIP-69) / edge3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants