Hello Dinesh Bhat, Todd Lipcon, Alexey Serbin, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/4180
to look at the new patch set (#5).
Change subject: tool: port kudu-admin to 'kudu table' and 'kudu tablet'
......................................................................
tool: port kudu-admin to 'kudu table' and 'kudu tablet'
The config change operations go to the revived 'tablet' mode, which is now
"operations against remote Kudu tablets". The table operations go to
'kudu table'.
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.h
M src/kudu/tools/tool_action_cluster.cc
M src/kudu/tools/tool_action_local_replica.cc
A src/kudu/tools/tool_action_table.cc
A src/kudu/tools/tool_action_tablet.cc
M src/kudu/tools/tool_main.cc
13 files changed, 457 insertions(+), 483 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/80/4180/5
--
To view, visit http://gerrit.cloudera.org:8080/4180
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Id6e96fc5b0106946f29a2faee8e2667be738b740
Gerrit-PatchSet: 5
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Dinesh Bhat <din...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mpe...@apache.org>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
