RNHTTR commented on issue #41036:
URL: https://github.com/apache/airflow/issues/41036#issuecomment-2284605199

   > Was something changed since my comment in [#30264 
(comment)](https://github.com/apache/airflow/issues/30264#issuecomment-1482028871)
 ? If not then I don't see what is the task here? If tasks marked as skipped 
due to dag run time out it means these tasks never started to run thus there is 
nothing to kill, as for the 2nd use case presented in #30264 the 
`on_skip_callback` was added to Airflow thus users can simply use that.
   
   Also, the `on_skipped_callback` is only called when an 
`AirflowSkipException` is raised. When a DAG run reaches its `dagrun_timeout`, 
the [task instances are skipped by the scheduler setting the TI's state to 
skipped](https://github.com/apache/airflow/blob/main/airflow/jobs/scheduler_job_runner.py#L1648),
 not by raising an `AirflowSkipException` for the TI.


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