wolfdn commented on PR #66705:
URL: https://github.com/apache/airflow/pull/66705#issuecomment-4427978512

   > This PR has a code-overlap with #66716 - can you check this as well?
   
   I just had a look at #66716 - it touches the same part of the code, but I 
think it solves a different issue. There are multiple things that could go 
wrong when the Triggerer passes back a task to the Worker:
   - Pod has been garbage collected by Kubernetes during the time window 
between Triggerer finishes and passes back Task to Airflow worker and Airflow 
worker picks back up the task → error handling improved by #66716
   - Triggerer returns task with error event, but Pod is still healthy and 
running → error handling improved by this PR (re-defer task)
   - Triggerer returns task with error event, but Pod has already completed 
successfully → error handling improved by this PR (treat as success)
   
   So I think both PRs are valid.


-- 
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]

Reply via email to