Hello Alexey Serbin, Ashwani Raina, Yingchun Lai, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/19411
to look at the new patch set (#4).
Change subject: [tools] add --tables flag to 'local_replica delete'
......................................................................
[tools] add --tables flag to 'local_replica delete'
This patch updates the 'local_replica delete' tool to allow deletion
multiple tablets by table name. Sometimes it is not convenient for
users to specify multiple tablet ids, but table names will be much
easier to get.
In order to support delete all tablets of specified tables, the
<tablet_ids> argument is updated to <tablet_id_patterns> which
support basic glob syntax. As a result, now we can not tell users
whether there is an illegal tablet id pattern well and truly, unless
all the specified tablet id patterns are tablet ids.
The default value of --tables flag is empty, which means all tables
will be included, so the tool can achieve same effect as before if
this flag is not specified.
A new test was added to verify the new functionality.
Change-Id: I2d862a715f05179b2e8def0d1cdfe58c32299329
---
M src/kudu/tools/kudu-tool-test.cc
M src/kudu/tools/tool_action_local_replica.cc
2 files changed, 102 insertions(+), 17 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/11/19411/4
--
To view, visit http://gerrit.cloudera.org:8080/19411
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I2d862a715f05179b2e8def0d1cdfe58c32299329
Gerrit-Change-Number: 19411
Gerrit-PatchSet: 4
Gerrit-Owner: Yifan Zhang <chinazhangyi...@163.com>
Gerrit-Reviewer: Alexey Serbin <ale...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ara...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Yifan Zhang <chinazhangyi...@163.com>
Gerrit-Reviewer: Yingchun Lai <acelyc1112...@gmail.com>
