potiuk commented on code in PR #58825:
URL: https://github.com/apache/airflow/pull/58825#discussion_r2582226300
##########
airflow-core/docs/authoring-and-scheduling/serializers.rst:
##########
@@ -87,7 +87,7 @@ Registered
from typing import TYPE_CHECKING
- from airflow.utils.module_loading import qualname
+ from airflow._shared.module_loading import qualname
Review Comment:
Yes. Same thought.
I was thinking that in airflow.sdk, the `_shared` should really be treated
as `private` and if we want to expose something outside we should make a
corresponding module that will be at `airflow.sdk.` level - just importing
things from `._shared` if we will think about making it available for Dag
Authors. This way we can also only expose what we **really** think should be
exposed rather than everything from the `shared` modules. WDYT?
--
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]
