uranusjr commented on code in PR #27710:
URL: https://github.com/apache/airflow/pull/27710#discussion_r1034343974
##########
airflow/cli/cli_parser.py:
##########
@@ -543,6 +543,11 @@ def string_lower_type(val):
help="Ignore depends_on_past dependencies (but respect upstream
dependencies)",
action="store_true",
)
+ARG_WAIT_FOR_PAST_DEPENDS_BEFORE_SKIPPING = Arg(
+ ("-W", "--wait-for-past-depends-before-skipping"),
+ help="Wait for past dependencies before skipping the task when
--ignore-depends-on-past is not set",
+ action="store_true",
+)
Review Comment:
I wonder if we should merge this and `--ignore-depends-on-past` into one
flag, say a `--depends-on-past` option that allows three possible values
`check` (default), `ignore`, and `wait`. This can be done in a subsequent,
separate PR, but we should start considering the design.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
