-
Notifications
You must be signed in to change notification settings - Fork 14.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Local and Sequential Executors to Doc #8084
Conversation
Codecov Report
@@ Coverage Diff @@
## master #8084 +/- ##
==========================================
- Coverage 87.28% 87.01% -0.27%
==========================================
Files 937 937
Lines 45520 45549 +29
==========================================
- Hits 39731 39634 -97
- Misses 5789 5915 +126
Continue to review full report at Codecov.
|
… Executor section
@mik-laj Thanks for the suggestions and they are fixed. PTAL :) |
locally, into just one `LocalExecutor` with multiple modes. | ||
LocalExecutor | ||
|
||
.. seealso:: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add similar annotations for other executors?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done!
…docs to Executor section
Co-Authored-By: Kaxil Naik <kaxilnaik@gmail.com>
@kaxil just removed the leading empty lines. Thanks for pointing it out. |
(cherry picked from commit ee12529)
(cherry picked from commit ee12529)
(cherry picked from commit ee12529)
Local and Sequential executors are currently missing the
Executor
section of the doc.Make sure to mark the boxes below before creating PR: [x]
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.
Read the Pull Request Guidelines for more information.