-
Notifications
You must be signed in to change notification settings - Fork 847
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
Turn debug messages into DEBUG1 #2443
Conversation
6ad2577
to
3cb7bc3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think there are probably more candidates for this treatment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FWIW, I think these log statements can be removed altogether. They served some purpose as NOTICEs to users, but as DEBUG1 messages I don't think they provide any useful information. If one wants to validate that these indexes exists, one need to check the table they get added to.
Yeah, but if I take to many at a time, there is going to be a lot of discussions about some of them, which will just block progress. Better take it in smaller and more clear batches. |
I'm fine with removing them altogether, but I thought moving them to |
df6bfa1
to
f495dc1
Compare
We have some debug messages that are printed as notices, but are more suitable to have at `DEBUG1` level. This commit removes a notice about indexes being added and turns it into a `DEBUG1` notice.
f495dc1
to
ae57da1
Compare
Codecov Report
@@ Coverage Diff @@
## master #2443 +/- ##
==========================================
+ Coverage 89.99% 90.11% +0.11%
==========================================
Files 213 213
Lines 34406 34356 -50
==========================================
- Hits 30964 30960 -4
+ Misses 3442 3396 -46
Continue to review full report at Codecov.
|
We have some debug messages that are printed as notices, but are more
suitable to have at
DEBUG1
level. This commit removes a notice aboutindexes being added and turns it into a
DEBUG1
notice.