jason810496 commented on PR #56869:
URL: https://github.com/apache/airflow/pull/56869#issuecomment-3420831250

   > This looks to me issue with api, the tests here explicitly we are 
validating s3 log source path returning correctly or not.
   > 
   > To me i am not sure why the api returning local log location? instead of 
s3 source
   
   Good point. I found why sometime we will get local source path instead of s3 
path:
   
   The `FileTaskHandler` will try to read from remote log, and if we are not 
able to get remote log (e.g. still uploading), we will try to read from local.
   
   
https://github.com/apache/airflow/blob/7f2704bc19ea6d6da41ea26232a16986353b090b/airflow-core/src/airflow/utils/log/file_task_handler.py#L607-L646
   
   It's not an issue in API but a fallback feature for fetching logs.


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