houqp commented on a change in pull request #7644: [AIRFLOW-7003] Lazy load all
plguins
URL: https://github.com/apache/airflow/pull/7644#discussion_r390008524
##########
File path: airflow/serialization/serialized_objects.py
##########
@@ -326,14 +326,15 @@ def serialize_operator(cls, op: BaseOperator) -> dict:
def deserialize_operator(cls, encoded_op: Dict[str, Any]) -> BaseOperator:
"""Deserializes an operator from a JSON object.
"""
- from airflow.plugins_manager import operator_extra_links
+ from airflow import plugins_manager
Review comment:
not related to your change, but i am just curious: do you know what's the
reason to do a lazy import for plugins_manager here as well as in
`_deserialize_operator_extra_links`?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
