[GitHub] [airflow] kazanzhy commented on pull request #20044: Add ShortCircuitOperator configurability for respecting downstream trigger rules

[GitBox <gi...@apache.org>]

kazanzhy commented on pull request #20044:
URL: https://github.com/apache/airflow/pull/20044#issuecomment-986347439


   Very confusing feature, I think.
   I just trying to figure out how it will work. 
   Let's imagine the next DAG:
   ```
   shortcircuit >> [task_1, task_2]
   [task_1, task_2] >> task_3
   ```
   where `task_3` has trigger rule 'all_done' because I assume that both `task_1` and `task_2` might fail.
   So if I pass `ignore_downstream_trigger_rules=False` in `shortcircuit` then `task_3` will run any time. Am I right?
   
   Btw, I think commits should be squashed.


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