luc-pimentel commented on issue #76170: URL: https://github.com/apache/airflow/issues/76170#issuecomment-6559226188
> 4\. [@luc-pimentel](https://github.com/luc-pimentel) thanks for the clear write-up and the root-cause pointer. I have a fix ready that matches the behaviour you proposed. If you agree, I'll open the PR right away. > With the fix, `clean >> Label("rows") >> load` and `clean << Label("rows") << load` give the same dependencies as the unlabelled edge and as `set_downstream(..., Label(...))`. Labels on a whole TaskGroup (the [#17469](https://github.com/apache/airflow/issues/17469) case) behave as before. Yes, go ahead. That is the behaviour the report asks for, but whether it is the intended one is the maintainers' call, and an open PR is the quickest way to get that answer in my opinion. The existing `test_task_group_edge_modifier_chain` only covers a label pointing at a whole group, so a test for a labelled edge from a task inside a group is worth adding. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
