Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15275
Change subject: [clock] record maximum error in microseconds ...................................................................... [clock] record maximum error in microseconds AWS/EC2 cloud instances' dedicated NTP servers report sub-millisecond root distance and dispersion, so it makes sense to switch true time maximum error histogram into microseconds. [root@aws-centos]# chronyc sources 210 Number of sources = 1 MS Name/IP address Stratum Poll Reach LastRx Last sample =============================================================================== ^* 169.254.169.123 3 7 377 107 +25us[ +30us] +/- 456us [root@aws-centos]# chronyc tracking Reference ID : A9FEA97B (169.254.169.123) Stratum : 4 Ref time (UTC) : Sun Feb 23 06:21:05 2020 System time : 0.000003886 seconds fast of NTP time Last offset : +0.000002831 seconds RMS offset : 0.000017730 seconds Frequency : 30.479 ppm fast Residual freq : +0.001 ppm Skew : 0.051 ppm Root delay : 0.000272366 seconds Root dispersion : 0.000380717 seconds Update interval : 128.9 seconds Leap status : Normal In addition, as a follow-up to the recent series of patches in the context of KUDU-3048, I also added an extra metric gauge to track latest maximum error computed by the built-in NTP client. I also renamed 'builtin_ntp_walltime' into 'builtin_ntp_time' for brevity. This is a follow-up to 7d9d7009 and 6371ffdb5. Change-Id: Idaa950fd3dff3e2a4cedbf5ae5b93b49f7b9465c --- M src/kudu/clock/builtin_ntp.cc M src/kudu/clock/builtin_ntp.h M src/kudu/clock/hybrid_clock.cc 3 files changed, 30 insertions(+), 11 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/75/15275/1 -- To view, visit http://gerrit.cloudera.org:8080/15275 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Idaa950fd3dff3e2a4cedbf5ae5b93b49f7b9465c Gerrit-Change-Number: 15275 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
