Hello Tidy Bot, Attila Bukor, Kudu Jenkins, Andrew Wong, Todd Lipcon,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9881
to look at the new patch set (#3).
Change subject: [tools] ksck improvements [1/n]: Eliminate KsckMaster class
......................................................................
[tools] ksck improvements [1/n]: Eliminate KsckMaster class
ksck has a KsckCluster and a KsckMaster class. The KsckCluster class's
methods more-or-less map one-to-one with KsckMaster class methods.
The KsckMaster class represents the collective masters of a cluster,
not an individual master.
This patch combines the two classes into a KsckCluster class. This
class is the base class for a MockKsckCluster class and a
RemoteKsckCluster class, for tests and "real life", respectively.
This is just refactoring. There are no functional changes in this patch.
Change-Id: I21f9e244b6ba10e11327c744abc1ac72a9b2ab1c
---
M src/kudu/client/schema.h
M src/kudu/integration-tests/cluster_verifier.cc
M src/kudu/tools/ksck-test.cc
M src/kudu/tools/ksck.cc
M src/kudu/tools/ksck.h
M src/kudu/tools/ksck_remote-test.cc
M src/kudu/tools/ksck_remote.cc
M src/kudu/tools/ksck_remote.h
M src/kudu/tools/tool_action_cluster.cc
M src/kudu/tools/tool_action_tablet.cc
10 files changed, 118 insertions(+), 182 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/81/9881/3
--
To view, visit http://gerrit.cloudera.org:8080/9881
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I21f9e244b6ba10e11327c744abc1ac72a9b2ab1c
Gerrit-Change-Number: 9881
Gerrit-PatchSet: 3
Gerrit-Owner: Will Berkeley <wdberke...@gmail.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <abu...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com>
