Hello Kudu Jenkins, Adar Dembo, Volodymyr Verovkin, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/15216 to look at the new patch set (#3). 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 --- 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(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/16/15216/3 -- 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: newpatchset Gerrit-Change-Id: I251c538c65bfc75a882218c2d946c795e69bcbbe Gerrit-Change-Number: 15216 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Volodymyr Verovkin <verjov...@cloudera.com>