jscheffl commented on PR #63282: URL: https://github.com/apache/airflow/pull/63282#issuecomment-4034535342
Thanks, this PR looks much better than #63176 which I reviewed yesterday. Also suffering from the same in our environment. You have now implemented the log fetching via a fallback, have you considered always calling to retrieve raw bytes and converting to unicode as you do in the fallback? Would be less code and in case of error just a single call. Otherwise, just a nit in static checks, in the Github logs you see how to run locally, in any kind of question please check contributors guide. -- 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]
