Lee-W commented on code in PR #30227:
URL: https://github.com/apache/airflow/pull/30227#discussion_r1150462596
##########
docs/apache-airflow-providers-dbt-cloud/operators.rst:
##########
@@ -108,6 +115,7 @@ Use the
:class:`~airflow.providers.dbt.cloud.sensors.dbt.DbtCloudJobRunAsyncSens
status of a dbt Cloud job run asynchronously. This sensor will free up the
worker slots since
polling for job status happens on the Airflow triggerer, leading to efficient
utilization
of resources within Airflow.
+:class:`~airflow.providers.dbt.cloud.sensors.dbt.DbtCloudJobRunAsyncSensor` is
deprecated and will be removed in a future release. Please use
:class:`~airflow.providers.dbt.cloud.sensors.dbt.DbtCloudJobRunSensor` and use
the deferrable mode in that operator.
Review Comment:
sure! I just update it. I'm not sure whether we can remove the code snippet.
If my memory serves me right, it won't pass the CI.
##########
docs/apache-airflow-providers-dbt-cloud/operators.rst:
##########
@@ -69,6 +69,14 @@ referenced within the ``default_args`` of the example DAG.
:start-after: [START howto_operator_dbt_cloud_run_job]
:end-before: [END howto_operator_dbt_cloud_run_job]
+Also you can use deferrable mode in this operator if you would like to free up
the worker slots while the sensor is running.
Review Comment:
Just fix it. Thanks!
--
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]
