Hello Kudu Jenkins, Adar Dembo,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/14317
to look at the new patch set (#2).
Change subject: [clock] change in the semantics of TimeService::Init()
......................................................................
[clock] change in the semantics of TimeService::Init()
This patch introduces a change in the semantics of the TimeService's
initialization method Init(). With this patch, TimeService::Init()
no longer waits for the clock synchronisation to happen for NTP-based
time services. Users of the TimeService interface should rely on
TimeService::WalltimeWithError() to get the status of the clock
synchronisation after calling TimeService::Init().
This is a follow-up to e110cb88b145edb6d536ae0806e02736f5bf51e5.
Change-Id: Ie7245b2cd2c3001a508235f9e539d7202d3ca994
---
M src/kudu/clock/hybrid_clock.cc
M src/kudu/clock/hybrid_clock.h
M src/kudu/clock/mock_ntp.cc
M src/kudu/clock/system_ntp.cc
M src/kudu/clock/system_ntp.h
M src/kudu/clock/system_unsync_time.cc
M src/kudu/clock/time_service.h
7 files changed, 66 insertions(+), 51 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/17/14317/2
--
To view, visit http://gerrit.cloudera.org:8080/14317
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie7245b2cd2c3001a508235f9e539d7202d3ca994
Gerrit-Change-Number: 14317
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
