vincbeck commented on code in PR #40468:
URL: https://github.com/apache/airflow/pull/40468#discussion_r1664610169


##########
airflow/providers/amazon/aws/executors/ecs/ecs_executor.py:
##########
@@ -320,10 +320,11 @@ def __handle_failed_task(self, task_arn: str, reason: 
str):
                 )
             )
         else:
-            self.log.error(
+            self.task_context_logger.error(

Review Comment:
   > is this something different from task retries? something like executor 
submit tries?
   
   Yes, it is a retry mechanism if the ECS task fails itself (thus, not able to 
start the Airflow task). If the ECS task succeeds and then the Airflow task 
fails, then the task retry comes in play



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