eladkal commented on a change in pull request #17144:
URL: https://github.com/apache/airflow/pull/17144#discussion_r674608885
##########
File path: docs/apache-airflow/concepts/dags.rst
##########
@@ -337,7 +337,7 @@ However, this is just the default behaviour, and you can
control it using the ``
* ``none_failed``: All upstream tasks have not ``failed`` or
``upstream_failed`` - that is, all upstream tasks have succeeded or been skipped
* ``none_failed_or_skipped``: All upstream tasks have not ``failed`` or
``upstream_failed``, and at least one upstream task has succeeded.
* ``none_skipped``: No upstream task is in a ``skipped`` state - that is, all
upstream tasks are in a ``success``, ``failed``, or ``upstream_failed`` state
-* ``dummy``: No dependencies at all, run this task at any time
+* ``null``: No dependencies at all, run this task at any time
Review comment:
I'm happy with any of the names suggested.
We had `dummy` trigger rule and `DummyOperator` but we don't have to choose
the same name for both we can have `None` trigger rule and `NullOperator`.
I can do the change now or wait for more thoughts.. I assume there will be
more opinions on that issue.
--
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]
