-
Notifications
You must be signed in to change notification settings - Fork 985
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 Deposit Processing Logs #13611
Conversation
"merkleTreeIndex": index, | ||
}).Debug("Deposit registered from deposit contract") | ||
// Log the deposit received periodically | ||
if index%depositLoggingInterval == 0 { |
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.
You need to deal with the case when depositLoggingInterval is zero
validDepositsCount.Inc() | ||
// Notify users what is going on, from time to time. | ||
if !s.chainStartData.Chainstarted { | ||
deposits := len(s.chainStartData.ChainstartDeposits) | ||
if deposits%512 == 0 { | ||
if deposits%depositLoggingInterval == 0 { |
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.
Sand here
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.
Discussed in slack
What type of PR is this?
Cleanup
What does this PR do? Why is it needed?
This PR cleans up our deposit processing logs with an interval of 1024.
Which issues(s) does this PR fix?
Fixes #13548
Other notes for review