Alexey Serbin has uploaded this change for review. (
http://gerrit.cloudera.org:8080/15215
Change subject: [clock] update on Clock interface
......................................................................
[clock] update on Clock interface
This patch re-factors Clock-related classes:
* removed Clock::RegisterMetrics() method
* HybridClock constructor requires metric entity
* LogicalClock constructor accepts metric entity as optional
second parameter
* LogicalClock constructor is now public
* LogicalClock::CreateStartingAt() is gone
I also did other minor re-factoring, partially due to warnings reported
by ClangTidy on the code I touched.
The motivation for this change is to prepare for follow-up changelists
addressing KUDU-3048 (adding clock metrics for better observability).
Change-Id: Ic4c1944d54bf50e54c06c12e2fb9e57fc452b877
---
M src/kudu/clock/clock.h
M src/kudu/clock/hybrid_clock-test.cc
M src/kudu/clock/hybrid_clock.cc
M src/kudu/clock/hybrid_clock.h
M src/kudu/clock/logical_clock-test.cc
M src/kudu/clock/logical_clock.cc
M src/kudu/clock/logical_clock.h
M src/kudu/consensus/consensus_peers-test.cc
M src/kudu/consensus/consensus_queue-test.cc
M src/kudu/consensus/log-test-base.h
M src/kudu/consensus/log_cache-test.cc
M src/kudu/consensus/raft_consensus_quorum-test.cc
M src/kudu/consensus/time_manager-test.cc
M src/kudu/integration-tests/ts_recovery-itest.cc
M src/kudu/server/server_base.cc
M src/kudu/tablet/compaction-test.cc
M src/kudu/tablet/deltamemstore-test.cc
M src/kudu/tablet/diskrowset-test-base.h
M src/kudu/tablet/diskrowset-test.cc
M src/kudu/tablet/memrowset-test.cc
M src/kudu/tablet/mvcc-test.cc
M src/kudu/tablet/mvcc.h
M src/kudu/tablet/tablet-harness.h
M src/kudu/tablet/tablet-test-util.h
M src/kudu/tools/tool_action_perf.cc
25 files changed, 335 insertions(+), 314 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/15/15215/1
--
To view, visit http://gerrit.cloudera.org:8080/15215
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic4c1944d54bf50e54c06c12e2fb9e57fc452b877
Gerrit-Change-Number: 15215
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>
