subhramit commented on code in PR #70795:
URL: https://github.com/apache/airflow/pull/70795#discussion_r4085183218


##########
airflow-core/src/airflow/jobs/scheduler_job_runner.py:
##########
@@ -2948,7 +2948,7 @@ def _schedule_dag_run(
                 default=None,
             )
             for task_instance in unfinished_task_instances:
-                task_instance.state = TaskInstanceState.SKIPPED
+                task_instance.state = TaskInstanceState.FAILED

Review Comment:
   Correct me if I'm wrong, but won't this affect every DAG run timeout 
unconditionally and not only mapped tasks?
   That would cause every unfinished TI of every timed-out run become FAILED, 
including TIs that never started.



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