kokorin commented on PR #26105:
URL: https://github.com/apache/airflow/pull/26105#issuecomment-1234736082

   @potiuk after investigation it looks like the root of the problem is in 
[FileTaskHandler](https://github.com/apache/airflow/blob/main/airflow/utils/log/file_task_handler.py#L115)
 as it writes logs to paths containing dates in [ISO 
8601format](https://en.wikipedia.org/wiki/ISO_8601). But ISO 8601format won't 
work on Windows.
   
   [Rewrite Breeze and CI in Python 
](https://github.com/apache/airflow/projects/13) project has no issue to use 
another date format to make log directory structure Windows-compatible. 
Probably it's because the project aimed at Breeze rewriting, not on been able 
to run Airflow on Windows.
   
   Changing `FileTaskHandler` is out of scope of this PR and the only way I see 
is to add a class extending `FileTaskHandler` and overwriting log directory 
structure with Windows compatible. After that `DEFAULT_LOGGING_CONFIG` in test 
should be adjusted to use customized `FileTaskHandler`.
   
   The class (customized `FileTaskHandler`) itself is not a problem, but I'm 
not an expert in Python and newbie Airflow developer. Could you tell me how to 
make Airflow/Breeze to load customized `FileTaskHandler`?


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