-
Notifications
You must be signed in to change notification settings - Fork 232
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
Clean up debug log level by moving many frequent logs to trace level #2004
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #2004 +/- ##
==========================================
- Coverage 59.73% 59.64% -0.09%
==========================================
Files 663 663
Lines 31552 31552
==========================================
- Hits 18847 18820 -27
- Misses 10025 10045 +20
- Partials 2680 2687 +7
Continue to review full report at Codecov.
|
app/protocol/flowcontext/blocks.go
Outdated
|
||
unorphaningResults, err := f.UnorphanBlocks(block) | ||
if err != nil { | ||
return err | ||
} | ||
|
||
log.Debugf("OnNewBlock: block %s unorphaned %d blocks", hash, len(unorphaningResults)) | ||
log.Tracef("OnNewBlock: block %s unorphaned %d blocks", hash, len(unorphaningResults)) |
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.
Consider changing to Debugf
@@ -79,11 +79,11 @@ func (flow *sendVersionFlow) start() error { | |||
} | |||
|
|||
// Wait for verack | |||
log.Debugf("Waiting for verack") | |||
log.Tracef("Waiting for verack") |
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.
Consider changing to Debugf
_, err = flow.incomingRoute.DequeueWithTimeout(common.DefaultTimeout) | ||
if err != nil { | ||
return err | ||
} | ||
log.Debugf("Got verack") | ||
log.Tracef("Got verack") |
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.
Consider changing to Debugf
} | ||
|
||
// Update reindex root data store | ||
rt.stageReindexRoot(stagingArea, newReindexRoot) | ||
log.Debugf("Updated the reindex root to %s", newReindexRoot) | ||
log.Tracef("Updated the reindex root to %s", newReindexRoot) |
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.
Maybe change to Debugf?
infrastructure/logger/utils.go
Outdated
} | ||
} | ||
|
||
// LogMemoryStats logs memory stats for `functionName` | ||
func LogMemoryStats(log *Logger, functionName string) { | ||
log.Debug(NewLogClosure(func() string { | ||
log.Trace(NewLogClosure(func() string { |
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.
Why?
No description provided.