insomnes commented on issue #45529:
URL: https://github.com/apache/airflow/issues/45529#issuecomment-2581303899

   I am talking about this 
[line](https://github.com/apache/airflow/blob/main/airflow/utils/log/file_task_handler.py#L362):
   ```python
           if ti.state == TaskInstanceState.RUNNING:
               response = self._executor_get_task_log(ti, try_number)
   
   ...
   
       @cached_property
       def _executor_get_task_log(self) -> Callable[[TaskInstance, int], 
tuple[list[str], list[str]]]:
           """This cached property avoids loading executor repeatedly."""
           executor = ExecutorLoader.get_default_executor()
           return executor.get_task_log
   ```
   
   I'm not very familiar with `ExecutorLoader` and maybe the real culprit is 
`@cached_property` usage, but this needs some rework for multiple executors 
support


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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to