yaming-github commented on issue #54145:
URL: https://github.com/apache/airflow/issues/54145#issuecomment-3166633014

   > [@yaming-github](https://github.com/yaming-github) How do you propose 
fixing this?
   
   Hi @ashb thanks for replying. Several things in my mind:
   
   1. We switched to use structlog as logging engine from this pr 
https://github.com/apache/airflow/pull/43893, but 
structlog.processors.CallsiteParameterAdder was not added as part of the 
processor chains. So I think we could start with adding that processor and test 
first.
   2. log_format in the config file was not respected anymore since 3+, so I 
think that could be a problem. If we add those call site parameters in the 
event disctionary, I think we will be able to fix that.
   3. Based on 1 and 2, we might also need to change how we render the log at 
frontend side. I haven't looked into that, but I guess should not be a bug 
issue.
   
   What do you think?


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