dheerajturaga commented on PR #72822: URL: https://github.com/apache/airflow/pull/72822#issuecomment-5623219156
> Agreed. My view right now is that this shouldn't be allowed and should be treated as an Invalid dag. > > I know there is _technically_ no cycle, but with some of the changes coming up (Task Loops, Dynamic Task Groups etc) and previous requests for "retry this task group" or "wait for task group completion" would be undefined behavior if we allow this. Fair point. I see how this can trip up the other features around Task Groups. I think we should catch this pattern at dag parse. This allows us to catch and fix things upfront before we discover post upgrade. I can raise another PR -- 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]
