-
Notifications
You must be signed in to change notification settings - Fork 14.2k
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
[AIRFLOW-5398] Update contrib example DAGs to context manager #5998
Conversation
This change looks fine, but it's there a reason why just these example dags are updated rather than all? I feel there was an existing issue to update all dags to use the context manager. |
@ashb I worked with these specific examples didn't check the other ones |
@ashb Rebased again to solve conflicts |
@eladkal Can you extract an example from docs/howto/operator/papermill.rst file and improve it also? |
@mik-laj Done |
…#5998) - replaces the explicit assignment of a dag to a task to be implicit via context manager - changes task dependencies to be defined via bit shifting operator instead of set_upstream/set_downstream - update docs for papermill to extract the code example from the code (cherry picked from commit 9c52848)
…#5998) - replaces the explicit assignment of a dag to a task to be implicit via context manager - changes task dependencies to be defined via bit shifting operator instead of set_upstream/set_downstream - update docs for papermill to extract the code example from the code (cherry picked from commit 9c52848)
Make sure you have checked all steps below.
Jira
Description
Update example DAGs of EMR to context manager and bitshift composition.
Tests
Commits
Documentation
Code Quality
flake8