Added documentation explaining difference deferred vs async operators#63500
Added documentation explaining difference deferred vs async operators#63500dabla wants to merge 19 commits intoapache:mainfrom
Conversation
…async operators with examples and explaining what the difference is
jscheffl
left a comment
There was a problem hiding this comment.
Looks okay for me, some proposals as feedback.
|
Thanks for the thorough review @jscheffl will fix the remarks. |
kaxil
left a comment
There was a problem hiding this comment.
Doc review of the new "Deferred vs Async Operators" page. The content fills a real gap — users need this guidance. Most comments are about broken cross-references (build failures) and making the examples more complete for users.
|
I feel we should also discuss when you should not use one of the other. For example
Also, the mention about existing deferrable operator can be a bit misleading IMO. If your use case is better for defer, and there’s no existing deferrable operator available, your best course would be to write one yourself, not to use async. It should depend more about the use case, not what Airflow has available. |
ecbd95d to
8638713
Compare
jscheffl
left a comment
There was a problem hiding this comment.
Looks good for me now!
Feedback by others welcome though.
Added documentation explaining difference deferred vs async operators, related to the PR #60268 which added native async support for PythonOperators.
Was generative AI tooling used to co-author this PR?
{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.