anishgirianish commented on code in PR #60855:
URL: https://github.com/apache/airflow/pull/60855#discussion_r2791189749
##########
task-sdk/src/airflow/sdk/execution_time/supervisor.py:
##########
@@ -2088,6 +2102,10 @@ def supervise(
final_state=process.final_state,
)
return exit_code
+ except TaskAlreadyRunningError:
+ # Let the executor handle this (e.g., Celery will ignore it)
+ log.info("Exiting due to broker redelivery",
task_instance_id=str(ti.id))
Review Comment:
Thanks for catching that. Replaced all broker/redelivery references in the
task-sdk with executor-agnostic language:
- Log: "Task is already running elsewhere, exiting."
- Comment: "Let the executor handle this."
- TaskAlreadyRunningError docstring: "Raised when a task is already
running on another worker."
The celery provider still uses "redelivered message" in its own log since
that's the appropriate context there.
--
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]
