Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14301 )
Change subject: [clock] move clock sync wait into HybridClock::Init() ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/14301/1/src/kudu/clock/hybrid_clock.cc File src/kudu/clock/hybrid_clock.cc: http://gerrit.cloudera.org:8080/#/c/14301/1/src/kudu/clock/hybrid_clock.cc@143 PS1, Line 143: s = time_service_->Init(); : if (!s.IsServiceUnavailable()) { : break; : } > Should probably doc this aspect of Init() in time_service.h since it's part Done -- To view, visit http://gerrit.cloudera.org:8080/14301 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I4bc5d3160a68932a31cde5ff221f74452bb14a21 Gerrit-Change-Number: 14301 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 25 Sep 2019 18:48:10 +0000 Gerrit-HasComments: Yes
