You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/03/09 23:04:40 UTC

[GitHub] [airflow] houqp commented on a change in pull request #7644: [AIRFLOW-7003] Lazy load all plguins

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:
users@infra.apache.org


With regards,
Apache Git Services