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>

Reply via email to