potiuk commented on pull request #11373:
URL: https://github.com/apache/airflow/pull/11373#issuecomment-707277489


   > One heisen test failed in a slightly odd way:
   > 
   > ```
   > self = <airflow.executors.dask_executor.DaskExecutor object at 
0x7efe33cd1090>
   > 
   >     def end(self) -> None:
   >         if not self.client:
   >             raise AirflowException(NOT_STARTED_MESSAGE)
   >         if not self.futures:
   > >           raise AirflowException(NOT_STARTED_MESSAGE)
   > E           airflow.exceptions.AirflowException: The executor should be 
started first!
   > 
   > airflow/executors/dask_executor.py:113: AirflowException
   > ```
   > 
   > Do we think it's safe to merge anyway?
   
   You can run the tests easily now for the whole group :) . LEt me do it for 
you


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


Reply via email to