You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/12/01 19:01:58 UTC

[GitHub] [airflow] eladkal commented on pull request #19361: Implement task-level include/exclude time filter

eladkal commented on pull request #19361:
URL: https://github.com/apache/airflow/pull/19361#issuecomment-983964535


   > BTW. An interesting use-case for pre-execute:
   > 
   > https://apache-airflow.slack.com/archives/CCQ7EGB1P/p1638380797413000?thread_ts=1638342676.385300&cid=CCQ7EGB1P
   > 
   > Someone wants to get notified mid-flight when the task is running too long. For me sounds like a perfect case for pre-execute calllable (and possibly we could also add a decorator to do the same) .
   
   I would say that this is something for SLA breach callback which is dedicated feature that eventually we will need to do. The SLA feature in Airflow works really weird.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org