myps6415 commented on PR #67592: URL: https://github.com/apache/airflow/pull/67592#issuecomment-5565703056
@samraj2k thanks for the ping — still active, no need to duplicate it. Approved and CI green; the only thing left is a semantics call on one open thread: https://github.com/apache/airflow/pull/67592#discussion_r3718746701 @kaxil that one's yours to settle. @henry3260 is happy to drop the `next_method` guard he originally asked for, and I lean the same way — one sample per queue wait. The deciding case for me is `start_from_trigger=True`: deferred straight from SCHEDULED, never queued before that, so the resume leg the guard skips is their only real wait and they emit nothing at all today. Confirm either way and I'll push the change, the test updates and a rebase in one go — the file has picked up a conflict since the last push. -- 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]
