[GitHub] [airflow] uranusjr commented on pull request #31987: Load just the first yielded event in the triggerer

["uranusjr (via GitHub)" <gi...@apache.org>]

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: commits-unsubscribe@airflow.apache.org

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