uranusjr commented on PR #31987: URL: https://github.com/apache/airflow/pull/31987#issuecomment-1667380195
I don’t have particular thoughts toward the analyses, but this change actually raises a big question in the interface design. If we always just break after the first yielded event, why do events need to be yielded in the first place anyway? Why is `run` on a trigger not just _returning_ an event instead? That would resolve the entire post-yield work discussion since it’s not possible to do things after a `return` in the first place. It seems to me if we do this, we should start re-design the trigger interface for 2.8. -- 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]
