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 (#5). 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 is 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/5 -- 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: 5 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>