APP-1547 Added sqlalchemy mapper to loquacious logger list#14292
Merged
Conversation
When starting up, sqlalchemy dumps a large quantity of data to the logs at info level. This is useful for debugging, but users do not need this under normal operation, so we increase the level from this logger to `WARNING`.
Contributor
Coverage reportClick to see where and how coverage changed
This report was generated by python-coverage-comment-action |
||||||||||||||||||||||||
all-hands-bot
approved these changes
May 4, 2026
all-hands-bot
left a comment
Collaborator
There was a problem hiding this comment.
🟢 Good taste - Simple, targeted configuration change that eliminates noisy startup logs
[RISK ASSESSMENT]
- [Overall PR]
⚠️ Risk Assessment: 🟢 LOW
This change only modifies logging verbosity for a single SQLAlchemy logger during startup. The scope is minimal, there are no breaking changes, no security implications, and the fix has been tested with clear before/after evidence. If any issues arise, this is easily reversible by removing one line.
VERDICT:
✅ Worth merging - Pragmatic solution to a real operational issue with strong evidence of effectiveness
KEY INSIGHT:
Proper log level management improves operator experience without changing any functional behavior.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
When starting up, sqlalchemy dumps a large quantity of data to the logs at info level. This is useful for debugging, but users do not need this under normal operation, so we increase the level from this logger to
WARNING.Issue Number
How to Test
Run the server and look at the logs. On main you'll see a large amount of data like this, which is conspicuously absent when the fix is applied:
Video/Screenshots
Type
Notes
To run this PR locally, use the following command:
GUI with Docker: