Yifan Zhang has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/19411 )
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 deleting 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 deleting all tablets of specified tables, the <tablet_ids> argument is updated to <tablet_id_patterns> which supports basic glob syntax. 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 was not specified. A new test is added to verify the new functionality. Change-Id: I2d862a715f05179b2e8def0d1cdfe58c32299329 Reviewed-on: http://gerrit.cloudera.org:8080/19411 Reviewed-by: Alexey Serbin <ale...@apache.org> Tested-by: Kudu Jenkins --- M src/kudu/tools/kudu-tool-test.cc M src/kudu/tools/tool_action_local_replica.cc 2 files changed, 207 insertions(+), 20 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved Kudu Jenkins: Verified -- 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: merged Gerrit-Change-Id: I2d862a715f05179b2e8def0d1cdfe58c32299329 Gerrit-Change-Number: 19411 Gerrit-PatchSet: 9 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> Gerrit-Reviewer: Ádám Bakai <aba...@cloudera.com>
