Hello Kudu Jenkins, Adar Dembo,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/15275
to look at the new patch set (#4).
Change subject: [clock] record maximum error in microseconds
......................................................................
[clock] record maximum error in microseconds
Dedicated NTP servers of AWS/EC2 cloud instances report sub-millisecond
root delay and dispersion (see below), so it makes sense to switch
true time maximum error histogram into microseconds.
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 time 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.
[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 8 377 106 +1063ns[+1441ns] +/- 466us
[root@aws-centos]# chronyc tracking
Reference ID : A9FEA97B (169.254.169.123)
Stratum : 4
Ref time (UTC) : Sun Feb 23 06:59:58 2020
System time : 0.000001781 seconds slow of NTP time
Last offset : +0.000000378 seconds
RMS offset : 0.000012111 seconds
Frequency : 30.588 ppm fast
Residual freq : -0.000 ppm
Skew : 0.019 ppm
Root delay : 0.000298246 seconds
Root dispersion : 0.000397126 seconds
Update interval : 260.0 seconds
Leap status : Normal
[root@aws-centos]# chronyc ntpdata 169.254.169.123
Remote address : 169.254.169.123 (A9FEA97B)
Remote port : 123
Local address : 10.65.14.145 (0A410E91)
Leap status : Normal
Version : 4
Mode : Server
Stratum : 3
Poll interval : 8 (256 seconds)
Precision : -18 (0.000003815 seconds)
Root delay : 0.000198 seconds
Root dispersion : 0.000305 seconds
Reference ID : A9FEA97A ()
Reference time : Sun Feb 23 06:55:06 2020
Offset : +0.000014476 seconds
Peer delay : 0.000101295 seconds
Peer dispersion : 0.000003855 seconds
Response time : 0.000015916 seconds
Jitter asymmetry: +0.00
NTP tests : 111 111 1111
Interleaved : No
Authenticated : No
TX timestamping : Kernel
RX timestamping : Kernel
Total TX : 712
Total RX : 712
Total valid RX : 712
Change-Id: Idaa950fd3dff3e2a4cedbf5ae5b93b49f7b9465c
---
M src/kudu/clock/builtin_ntp.cc
M src/kudu/clock/builtin_ntp.h
M src/kudu/clock/hybrid_clock-test.cc
M src/kudu/clock/hybrid_clock.cc
4 files changed, 49 insertions(+), 19 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/75/15275/4
--
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: newpatchset
Gerrit-Change-Id: Idaa950fd3dff3e2a4cedbf5ae5b93b49f7b9465c
Gerrit-Change-Number: 15275
Gerrit-PatchSet: 4
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
