Hello Mike Percy, Todd Lipcon,

I'd like you to do a code review.  Please visit

    http://gerrit.cloudera.org:8080/4180

to review the following change.

Change subject: tool: port kudu-admin to 'kudu cluster'
......................................................................

tool: port kudu-admin to 'kudu cluster'

In doing so, the ported functionality can now also work in multi-master
mode; previously, the ClusterAdminClient refused to accept multiple masters.

I opted to group the config change operations as a common mode rather than
an additional parameter. I think this makes them more discoverable.

Change-Id: Id6e96fc5b0106946f29a2faee8e2667be738b740
---
M build-support/dist_test.py
M docs/release_notes.adoc
M src/kudu/tools/CMakeLists.txt
M src/kudu/tools/kudu-admin-test.cc
D src/kudu/tools/kudu-admin.cc
M src/kudu/tools/kudu-tool-test.cc
M src/kudu/tools/kudu-ts-cli-test.cc
M src/kudu/tools/tool_action_cluster.cc
8 files changed, 327 insertions(+), 480 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/80/4180/1
-- 
To view, visit http://gerrit.cloudera.org:8080/4180
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id6e96fc5b0106946f29a2faee8e2667be738b740
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Mike Percy <mpe...@apache.org>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>

Reply via email to