Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14317 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/14317 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Adar Dembo <[email protected]> --- M src/kudu/clock/hybrid_clock.cc M src/kudu/clock/hybrid_clock.h M src/kudu/clock/mock_ntp.h M src/kudu/clock/system_ntp.cc M src/kudu/clock/system_ntp.h M src/kudu/clock/time_service.h 6 files changed, 51 insertions(+), 50 deletions(-) Approvals: Alexey Serbin: Verified Adar Dembo: Looks good to me, approved -- 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: merged Gerrit-Change-Id: Ie7245b2cd2c3001a508235f9e539d7202d3ca994 Gerrit-Change-Number: 14317 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)
