-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Do not log traceback when log format is json #7973
Conversation
Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide. |
@ChenyuLInx Change itself seems fine to me
We've no longer been cutting patches of v1.0 and v1.1 (since they've reached EOL) - let's discuss offline if there's a need to do this? (Update: started this internal Slack thread to discuss) |
* do not log traceback when log format is json * changelog (cherry picked from commit 283f716)
* do not log traceback when log format is json * changelog (cherry picked from commit 283f716)
* do not log traceback when log format is json * changelog (cherry picked from commit 283f716)
resolves #7972
This is only for 1.3 and pre versions of dbt-core.
When calling
logger.debug
withexc_info
set to True, logger will actually print out the formatted log to stdout(?). We want to avoid it when we set log_format toJSON
Will backport to 1.2 and 1.1 and 1.0 once this is merged.