Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15156
Change subject: [clock] simplify system time source ...................................................................... [clock] simplify system time source This changelist simplifies the SystemNtp time provider to use ntp_gettime() from the NTP kernel API instead of ntp_adjtime(), see [1] for details. As it turned out, the 'tolerance' field from timex structure is architecture-dependent constant and never changes: I verified that for Linux and FreeBSD kernels. In addition, this patch enables the 'system' time source for macOS HighSierra and newer, where the NTP kernel API is now supported. [1] http://man7.org/linux/man-pages/man3/ntp_gettime.3.html Change-Id: Ib197effb63e8f5e37fd828c88e705112bb7047ce --- M src/kudu/clock/CMakeLists.txt M src/kudu/clock/hybrid_clock.cc M src/kudu/clock/system_ntp.cc M src/kudu/clock/system_ntp.h 4 files changed, 38 insertions(+), 52 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/56/15156/1 -- To view, visit http://gerrit.cloudera.org:8080/15156 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ib197effb63e8f5e37fd828c88e705112bb7047ce Gerrit-Change-Number: 15156 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
