uranusjr commented on PR #32701:
URL: https://github.com/apache/airflow/pull/32701#issuecomment-1647662076

   Thanks for the very detailed explanation, that makes sense now! I think the 
fix is on the right track, but not at the right place. We have a separate dep 
that checks for this scenario (the current task needs to be expanded but is 
not) in `MappedTaskIsExpanded`, but that dep is currently only applied to 
individually mapped operators, not operators in a mapped task group. So I think 
the right fix should be to use that trigger dep for this instead, so when logic 
reaches TriggerRuleDep we can be sure the current task has been unmapped. I’ll 
submit a new PR for you to try.


-- 
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-unsubscr...@airflow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to