eladkal commented on code in PR #48854:
URL: https://github.com/apache/airflow/pull/48854#discussion_r2031454822
##########
airflow-core/docs/authoring-and-scheduling/event-scheduling.rst:
##########
@@ -43,7 +43,7 @@ SQS:
.. code-block:: python
from airflow.sdk import Asset, AssetWatcher
- from airflow.providers.common.msgq.triggers.msg_queue import
MessageQueueTrigger
+ from airflow.providers.common.messaging.triggers.msg_queue import
MessageQueueTrigger
Review Comment:
By doing so you are binding the doc update into core releases even though
the code sits in providers.
That normally invites troubles.
I didn't suggest to remove it, I suggested to shift it to the provider.
The core docs can refer to further read in providers.. we do that for k8s,
celery and others.
--
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]
