Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15216 )
Change subject: [clock] cleanup on accessing HybridClock::next_timestamp_ ...................................................................... [clock] cleanup on accessing HybridClock::next_timestamp_ Added DCHECK() into HybridClock::NowWithError() method to ensure that access to HybridClock::next_timestamp_ field is guarded. This patch contains other minor updates. Change-Id: I251c538c65bfc75a882218c2d946c795e69bcbbe Reviewed-on: http://gerrit.cloudera.org:8080/15216 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Alexey Serbin <[email protected]> --- M src/kudu/clock/clock.h M src/kudu/clock/hybrid_clock.cc M src/kudu/clock/hybrid_clock.h M src/kudu/clock/logical_clock.h 4 files changed, 158 insertions(+), 162 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Alexey Serbin: Verified -- To view, visit http://gerrit.cloudera.org:8080/15216 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I251c538c65bfc75a882218c2d946c795e69bcbbe Gerrit-Change-Number: 15216 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) Gerrit-Reviewer: Volodymyr Verovkin <[email protected]>
