uranusjr commented on code in PR #29406:
URL: https://github.com/apache/airflow/pull/29406#discussion_r1159329708


##########
airflow/models/dag.py:
##########
@@ -357,6 +358,9 @@ class DAG(LoggingMixin):
         Can be used as an HTTP link (for example the link to your Slack 
channel), or a mailto link.
         e.g: {"dag_owner": "https://airflow.apache.org/"}
     :param auto_register: Automatically register this DAG when it is used in a 
``with`` block
+    :param fail_stop: Fails currently running tasks when task in DAG fails
+        **Warning**: A fail stop dag can only have tasks with the default 
trigger rule ("all_success").
+        An exception will be thrown if any task in a fail stop dag has a non 
default trigger rule.

Review Comment:
   How does this render in documentation? I don’t think the newline would be 
rendered, and the end result would read weird without a period in the previous 
sentense.



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

Reply via email to