Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Volodymyr Verovkin,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/15249

to look at the new patch set (#5).

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
---
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(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/49/15249/5
--
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: newpatchset
Gerrit-Change-Id: Ia0a13d58c6e5da50ef7a98ae73de267ec4eab491
Gerrit-Change-Number: 15249
Gerrit-PatchSet: 5
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Volodymyr Verovkin <[email protected]>

Reply via email to