Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15249 )
Change subject: [clock] KUDU-3048 add builtin-NTP-specific metrics ...................................................................... [clock] KUDU-3048 add builtin-NTP-specific metrics Added metrics specific to the built-in NTP client: * info : difference between tracked true time and local wallclock * debug: most recently computed true time * debug: histogram on the maximum error of the computed true time For testing, I manually ran kudu-master and verified the newly introduced metrics are present and their values make sense. I also re-visited the level of recently introduced metrics in the HybridClock class. Change-Id: Ia0a13d58c6e5da50ef7a98ae73de267ec4eab491 Reviewed-on: http://gerrit.cloudera.org:8080/15249 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> --- M src/kudu/clock/builtin_ntp.cc M src/kudu/clock/builtin_ntp.h M src/kudu/clock/hybrid_clock.cc M src/kudu/clock/hybrid_clock.h M src/kudu/clock/ntp-test.cc 5 files changed, 135 insertions(+), 25 deletions(-) Approvals: Kudu Jenkins: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/15249 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ia0a13d58c6e5da50ef7a98ae73de267ec4eab491 Gerrit-Change-Number: 15249 Gerrit-PatchSet: 6 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Volodymyr Verovkin <[email protected]>
