Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/10151 to look at the new patch set (#5). Change subject: [WIP] [tools] ksck improvements [6/n]: Refactor printing ...................................................................... [WIP] [tools] ksck improvements [6/n]: Refactor printing This patch significantly refactors printing in ksck. It separates where checks are done from where results are printed. This separation will make it easier to provide machine-readable output and create tools based on the results of ksck checks, like auto-repair tools. There's also a couple of bonus changes: - Simplifies running all ksck checks and printing the results, as done by the ksck tool and ClusterVerifier. - Removes the --consensus flag. ksck is far less useful without it and it doesn't seem like having it brings any benefit. Change-Id: Id8de619996b6cd753e6a9c01b1b60810a873e609 --- M src/kudu/integration-tests/cluster_verifier.cc M src/kudu/tools/CMakeLists.txt 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 A src/kudu/tools/ksck_results.cc A src/kudu/tools/ksck_results.h M src/kudu/tools/kudu-admin-test.cc M src/kudu/tools/tool_action_cluster.cc M src/kudu/tools/tool_action_tablet.cc 11 files changed, 966 insertions(+), 781 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/51/10151/5 -- To view, visit http://gerrit.cloudera.org:8080/10151 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Id8de619996b6cd753e6a9c01b1b60810a873e609 Gerrit-Change-Number: 10151 Gerrit-PatchSet: 5 Gerrit-Owner: Will Berkeley <wdberke...@gmail.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com>