You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "canyang25 (via GitHub)" <gi...@apache.org> on 2026/10/05 01:51:10 UTC
[PR] Keep reschedule-mode sensors from failing on a stale executor success [airflow]
canyang25 opened a new pull request, #74212:
URL: https://github.com/apache/airflow/pull/74212
A reschedule-mode sensor exits successfully between pokes. When the scheduler puts the task back to scheduled or queued before it drains that executor success, it treats the success as an external state change and fails the task.
closes: #71172
Yes, Generated-by: Cursor (Grok)
--
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
Re: [PR] Keep reschedule-mode sensors from failing on a stale executor success [airflow]
Posted by "canyang25 (via GitHub)" <gi...@apache.org>.
canyang25 commented on PR #74212:
URL: https://github.com/apache/airflow/pull/74212#issuecomment-6402400051
Hi, this PR was approved by @luc-pimentel, and the latest CI run has no failing checks. Would you mind taking a look at whether anything else is needed before merging? Thanks!
--
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
Re: [PR] Keep reschedule-mode sensors from failing on a stale executor success [airflow]
Posted by "canyang25 (via GitHub)" <gi...@apache.org>.
canyang25 commented on PR #74212:
URL: https://github.com/apache/airflow/pull/74212#issuecomment-6042245353
Thanks @luc-pimentel, good catch. Switched to list(session.scalars(...)) as you suggested and rebased onto main. The diagnostic test and the two new tests pass locally; CI is waiting on workflow approval.
--
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