-
Notifications
You must be signed in to change notification settings - Fork 301
Conversation
97c0d49
to
b7a318e
Compare
Co-authored-by: Oleksii Matiiasevych <lastperson@gmail.com>
@@ -163,13 +167,15 @@ func (l *listener) pollBlocks() error { | |||
l.log.Error("Failed to write to blockstore", "err", err) | |||
} | |||
|
|||
if l.metrics != nil { | |||
l.metrics.BlocksProcessed.Inc() |
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.
Just a Q, if we were to need to reprocess blocks for some reason this counter could theoretically be larger than the latest processed block metric right?
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.
Yup! The blocks processed counter will also reset to 0 on restarts.
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.
do you think it might make more sense to specify that this is the blocks processed in a given "session" then in the docs? (votes submitted as well)
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.
Not really, this is almost always the case with counters such as this
Changes
Closes: #?