nailo2c commented on PR #60384: URL: https://github.com/apache/airflow/pull/60384#issuecomment-3745650852
Thanks for the review, and sorry my approach didn't match the current logging architecture in Airflow 3. This is also a good opportunity for me to learn. To align with the architecture, you mentioned using a structlog processor. Does that typically sit within a `RemoteLogIO` implementation? (e.g. [CloudWatchRemoteLogIO](https://airflow.apache.org/docs/apache-airflow-providers-amazon/stable/_modules/airflow/providers/amazon/aws/log/cloudwatch_task_handler.html#CloudWatchRemoteLogIO)) Appreciate the guidance, this review helps a lot :) -- 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]
