Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/19449 )
Change subject: [clock] detect unsynchronized clock earlier ...................................................................... [clock] detect unsynchronized clock earlier This patch adds an extra check for the return code of the ntp_adjtime() call in SystemNtp::Init(). This is to allow for earlier detection of unsynchronized local clock conditions if using the "system" time source. Before this patch, if the local clock wasn't synchronized with NTP, that fact would be detected later upon first call to WalltimeWithError(). The latter usually happens when a Kudu server tries to find current time to generate a hybrid timestamp for an operation. This is a follow-up to e72208436a625391739217394c67d783e992367a. Change-Id: I32fd6ac1adfa04251e1e4b6237d650071e3b4dc9 Reviewed-on: http://gerrit.cloudera.org:8080/19449 Reviewed-by: Yifan Zhang <[email protected]> Reviewed-by: Yingchun Lai <[email protected]> Tested-by: Alexey Serbin <[email protected]> --- M src/kudu/clock/system_ntp.cc 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Yifan Zhang: Looks good to me, approved Yingchun Lai: Looks good to me, approved Alexey Serbin: Verified -- To view, visit http://gerrit.cloudera.org:8080/19449 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I32fd6ac1adfa04251e1e4b6237d650071e3b4dc9 Gerrit-Change-Number: 19449 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yifan Zhang <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]>
