Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/4148 to look at the new patch set (#2). Change subject: tool: split up action descriptions ...................................................................... tool: split up action descriptions With ksck we have a use case for "short" and "long" action descriptions: the former appears in mode help and action help, while the latter is only in action help. This patch splits action descriptions into "short" and "extra", refactoring all actions accordingly. While I was there I got rid of the label abstraction, which I felt was producing too much unfluent code. I also added some more tests to kudu-tool-test. Change-Id: Ie90b6228bba54575933fd9ac2f4f0bebf579ecd7 --- M src/kudu/tools/kudu-tool-test.cc M src/kudu/tools/tool_action.cc M src/kudu/tools/tool_action.h M src/kudu/tools/tool_action_cluster.cc M src/kudu/tools/tool_action_fs.cc M src/kudu/tools/tool_action_pbc.cc M src/kudu/tools/tool_action_tablet.cc M src/kudu/tools/tool_main.cc 8 files changed, 261 insertions(+), 97 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/48/4148/2 -- To view, visit http://gerrit.cloudera.org:8080/4148 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ie90b6228bba54575933fd9ac2f4f0bebf579ecd7 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org>