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/02/01 20:17:23 UTC

[GitHub] [airflow] mik-laj edited a comment on issue #7327: [AIRFLOW-6706] Lazy load operator extra links

mik-laj edited a comment on issue #7327: [AIRFLOW-6706] Lazy load operator extra links
URL: https://github.com/apache/airflow/pull/7327#issuecomment-581065459
 
 
   I think this is a completely different problem. I also think it is worth solving it, but it will require more time and testing.  If we call the integrate plugins method elsewhere, it will still load GCP and Qubole classes, when it is not needed. In this PR I want to focus only on one small problem and not change the way whole plugins mechanism work. 
   
   My change is not even related to the plugin loading mechanism at all. This only applies to operators that are in the core. It is a mistake that this code is in the same module. 
   
   BTW. I think that it will also be much easier to do when we drop plugin support for operators, sensors and hook. 

----------------------------------------------------------------
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