Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17582 )
Change subject: [clock] change clock source selection for 'auto' ...................................................................... Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/17582/3/src/kudu/clock/hybrid_clock.cc File src/kudu/clock/hybrid_clock.cc: http://gerrit.cloudera.org:8080/#/c/17582/3/src/kudu/clock/hybrid_clock.cc@476 PS3, Line 476: // Switch to the built-in NTP client unless the set of reference servers > One of the nice things about the current iteration of this is that the sele Yep, this iteration is deterministic in the terms of current configuration and doesn't depend much on network and other runtime issues. The expectation is that the built-in client has less strict requirements to synchronize because it's not needed to sync the system clock (assuming the set of the NTP servers is the same for ntpd/chronyd and the built-in client). With that, it might happen that the built-in NTP client is able to synchronize quite fast, but the system clock cannot synchronized for a long time even if NTP server is latched with the reference NTP servers (e.g., the ntpd is running with '-x' option and the original offset of the local time was several seconds). -- To view, visit http://gerrit.cloudera.org:8080/17582 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I72cb54c488b2aa2c73acd6e6e5f6e50dd5811175 Gerrit-Change-Number: 17582 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Comment-Date: Fri, 09 Jul 2021 22:59:49 +0000 Gerrit-HasComments: Yes
