[GitHub] [airflow] ashb commented on pull request #11373: Spend less time waiting for LocalTaskJob's subprocss process to finish

[GitBox <gi...@apache.org>]

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


   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?


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