You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "jedcunningham (via GitHub)" <gi...@apache.org> on 2023/03/09 23:10:49 UTC
[GitHub] [airflow] jedcunningham opened a new pull request, #30008: Don't separate setup/teardowns from normal task
jedcunningham opened a new pull request, #30008:
URL: https://github.com/apache/airflow/pull/30008
Instead of keeping setup and teardown tasks separate from the normal task list in Task Groups, we will put them all into children together. Attributes on the tasks are all we need to support the setup/teardowns feature, and keeping them as normal-task-like as possible is desirable.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] uranusjr commented on pull request #30008: Don't separate setup/teardowns from normal task
Posted by "uranusjr (via GitHub)" <gi...@apache.org>.
uranusjr commented on PR #30008:
URL: https://github.com/apache/airflow/pull/30008#issuecomment-1463376019
Are there existing usages of `children` in the code base that we may need to change?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] jedcunningham merged pull request #30008: Don't separate setup/teardowns from normal task
Posted by "jedcunningham (via GitHub)" <gi...@apache.org>.
jedcunningham merged PR #30008:
URL: https://github.com/apache/airflow/pull/30008
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] jedcunningham commented on pull request #30008: Don't separate setup/teardowns from normal task
Posted by "jedcunningham (via GitHub)" <gi...@apache.org>.
jedcunningham commented on PR #30008:
URL: https://github.com/apache/airflow/pull/30008#issuecomment-1464628346
We are already touching most (all?) of the sections that might care, like the graph, clearing behavior, and trigger rules. I don't know of other instances that will need to change.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org