feluelle commented on a change in pull request #6352: [AIRFLOW-5683] Add
propagate_skipped_state to SubDagOperator
URL: https://github.com/apache/airflow/pull/6352#discussion_r341975079
##########
File path: airflow/operators/subdag_operator.py
##########
@@ -33,6 +37,14 @@
from airflow.utils.state import State
+class SkippedStatePropagationOptions(Enum):
+ """
+ Available options for skipped state propagation of subdag's tasks to
parent dag tasks.
+ """
+ ALL_LEAVES = 1
Review comment:
Maybe it is better to have string values.
```
ALL_LEAVES = 'all_leaves'
ANY_LEAF = 'any_leaf'
```
WDYT?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
