-
Notifications
You must be signed in to change notification settings - Fork 250
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
gossip: Add save_contact_info_time
metric
#4114
base: master
Are you sure you want to change the base?
Conversation
5c2600c
to
52aef03
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.
looks good! just the one nit
gossip/src/cluster_info_metrics.rs
Outdated
( | ||
"save_contact_info_time", | ||
stats.save_contact_info_time.clear(), | ||
i64 | ||
), |
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.
nit: I would put this under cluster_info_stats2
since there are more timing metrics there and cluster_info_stats5
is more packet/count related
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.
Done
Add `save_contact_info_time` metric which measures the time spent by the `solGossip` thread on fetching and saving contact info.
52aef03
to
a180e52
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.
lgtm! thank you!!
Add
save_contact_info_time
metric which measures the time spent by thesolGossip
thread on fetching and saving contact info.