mik-laj commented on a change in pull request #6596: [AIRFLOW-6004] Untangle 
Executors class to avoid cyclic imports
URL: https://github.com/apache/airflow/pull/6596#discussion_r347165003
 
 

 ##########
 File path: airflow/executors/dask_executor.py
 ##########
 @@ -44,6 +42,7 @@ def __init__(self, cluster_address=None):
         super().__init__(parallelism=0)
 
     def start(self):
+        import distributed
 
 Review comment:
   After introduction suggestions from comment - 
https://github.com/apache/airflow/pull/6596/files#r347158027 , the problem will 
disappear. The dask_executor.py file will not be loaded if the executor is not 
selected, so the `distributed` library is probably not installed.

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

Reply via email to