syedahsn commented on code in PR #32029:
URL: https://github.com/apache/airflow/pull/32029#discussion_r1243168780


##########
docs/apache-airflow-providers-amazon/operators/emr/emr.rst:
##########
@@ -189,6 +189,11 @@ Wait on an Amazon EMR job flow state
 To monitor the state of an EMR job flow you can use
 :class:`~airflow.providers.amazon.aws.sensors.emr.EmrJobFlowSensor`.
 
+This sensor can be run in deferrable mode by passing ``deferrable=True`` as a 
parameter.
+Using ``deferrable`` mode will release worker slots and leads to efficient 
utilization of
+resources within Airflow cluster.However this mode will need the Airflow 
triggerer to be
+available in your deployment.
+

Review Comment:
   I don't think it make sense to have this generic information about the 
benefits of deferrable operators everywhere. This document should just let the 
user know that deferrable mode is available. We have other documentation that 
highlight the benefits of deferrable operators.



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