[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-21 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r382499946 ## File path: airflow/models/skipmixin.py ## @@ -88,13 +134,23 @@ def skip_all_except(self

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-20 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r381943188 ## File path: airflow/models/skipmixin.py ## @@ -65,13 +68,53 @@ def skip(self, dag_run, e

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-20 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r381942853 ## File path: airflow/models/skipmixin.py ## @@ -65,13 +68,53 @@ def skip(self, dag_run, e

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-20 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r381942951 ## File path: airflow/models/skipmixin.py ## @@ -65,13 +68,53 @@ def skip(self, dag_run, e

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-17 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r380217423 ## File path: airflow/models/skipmixin.py ## @@ -88,13 +131,23 @@ def skip_all_except(self

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-17 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r380219507 ## File path: tests/ti_deps/deps/test_trigger_rule_dep.py ## @@ -19,6 +19,8 @@ import uni

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-17 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r380121894 ## File path: airflow/ti_deps/deps/not_previously_skipped_dep.py ## @@ -0,0 +1,71 @@ +# +#

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-17 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r380119916 ## File path: airflow/models/skipmixin.py ## @@ -65,13 +61,44 @@ def skip(self, dag_run, e

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-17 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r380122906 ## File path: airflow/ti_deps/deps/not_previously_skipped_dep.py ## @@ -0,0 +1,71 @@ +# +#

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-17 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r380124893 ## File path: tests/ti_deps/deps/test_not_previously_skipped_dep.py ## @@ -0,0 +1,133 @@ +

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-17 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r380117315 ## File path: airflow/models/skipmixin.py ## @@ -16,42 +16,38 @@ # specific language gove

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-17 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r380117315 ## File path: airflow/models/skipmixin.py ## @@ -16,42 +16,38 @@ # specific language gove

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-14 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r379649516 ## File path: airflow/models/baseoperator.py ## @@ -650,6 +651,7 @@ def deps(self) -> Set[

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-14 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r379470709 ## File path: tests/operators/test_python.py ## @@ -396,7 +397,7 @@ def test_without_dag_r

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-14 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r379470137 ## File path: airflow/ti_deps/deps/branch_dep.py ## @@ -0,0 +1,63 @@ +# +# Licensed to the

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-14 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r379327408 ## File path: airflow/ti_deps/deps/branch_dep.py ## @@ -0,0 +1,63 @@ +# +# Licensed to the

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-14 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r379326946 ## File path: airflow/ti_deps/deps/branch_dep.py ## @@ -0,0 +1,63 @@ +# +# Licensed to the

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-14 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r379332710 ## File path: airflow/ti_deps/deps/branch_dep.py ## @@ -0,0 +1,63 @@ +# +# Licensed to the

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-14 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r379334969 ## File path: airflow/models/baseoperator.py ## @@ -650,6 +651,7 @@ def deps(self) -> Set[

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-14 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r379328108 ## File path: airflow/models/skipmixin.py ## @@ -98,3 +98,17 @@ def skip_all_except(self,

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-14 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r379329062 ## File path: airflow/operators/branch_operator.py ## @@ -48,4 +48,21 @@ def choose_branch

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-14 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r379333409 ## File path: airflow/ti_deps/deps/branch_dep.py ## @@ -0,0 +1,63 @@ +# +# Licensed to the

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-14 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r379333613 ## File path: tests/operators/test_python.py ## @@ -396,7 +397,7 @@ def test_without_dag_r

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-14 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r379324932 ## File path: airflow/ti_deps/deps/branch_dep.py ## @@ -0,0 +1,63 @@ +# +# Licensed to the

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-13 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r378787145 ## File path: airflow/ti_deps/deps/branch_dep.py ## @@ -0,0 +1,63 @@ +# +# Licensed to the

[GitHub] [airflow] ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared

2020-02-13 Thread GitBox
ashb commented on a change in pull request #7276: [AIRFLOW-5391] Do not run skipped tasks when they are cleared URL: https://github.com/apache/airflow/pull/7276#discussion_r378784642 ## File path: airflow/ti_deps/deps/branch_dep.py ## @@ -0,0 +1,63 @@ +# +# Licensed to the