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/11/02 19:03:01 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-958077223


   >    Something like `trigger_rule={"state": "all_success", "include_date": ...}` – ?
   
   I can understand the concept of trigger rule to support more than just check against state.
   I mentioned it also in:
   https://github.com/apache/airflow/pull/17576#issuecomment-916429784
   
   That said, I'm not sure I follow on the concept of include/exclude. It feels dangerous to me. For example if the task has external sensor in another waiting for it to be executed?


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