SakshamSinghal20 commented on PR #62258:
URL: https://github.com/apache/airflow/pull/62258#issuecomment-3949428139
> Looking at this more closely, this is not a valid way of logging
dictionaries.
>
> Since we use underneath stdlib logging, it will crash which is expected.
Running this:
>
> ```python
> import logging
>
> logging.basicConfig(level=logging.DEBUG)
> logger = logging.getLogger(__name__)
>
> logger.warning("Info message %s", {"a": 10})
> ```
>
> would crash too. The right way to log a dict would be:
`logger.warning("Info message %s", str({"a": 10}))`
Thank You @amoghrajesh for letting me know I will change this ASAP.
--
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]
