Ashwani Raina has posted comments on this change. ( http://gerrit.cloudera.org:8080/19411 )
Change subject: [tools] add --tables flag to 'local_replica delete' ...................................................................... Patch Set 3: (4 comments) http://gerrit.cloudera.org:8080/#/c/19411/3//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/19411/3//COMMIT_MSG@9 PS3, Line 9: allow for : deleting multiple tablets nit: allow deletion of multiple tablets http://gerrit.cloudera.org:8080/#/c/19411/3//COMMIT_MSG@11 PS3, Line 11: operators nit: users http://gerrit.cloudera.org:8080/#/c/19411/3/src/kudu/tools/kudu-tool-test.cc File src/kudu/tools/kudu-tool-test.cc: http://gerrit.cloudera.org:8080/#/c/19411/3/src/kudu/tools/kudu-tool-test.cc@4208 PS3, Line 4208: some specified tablet id patterns do not exist If tabletid is specified and should work just like a pattern, it would make more sense to say: "specified tablet id does not exist" http://gerrit.cloudera.org:8080/#/c/19411/3/src/kudu/tools/tool_action_local_replica.cc File src/kudu/tools/tool_action_local_replica.cc: http://gerrit.cloudera.org:8080/#/c/19411/3/src/kudu/tools/tool_action_local_replica.cc@1392 PS3, Line 1392: AddOptionalParameter("tables") I am not sure if I get this. Where are we storing table name from user in case one wants to specify? -- 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: comment Gerrit-Change-Id: I2d862a715f05179b2e8def0d1cdfe58c32299329 Gerrit-Change-Number: 19411 Gerrit-PatchSet: 3 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-Comment-Date: Wed, 11 Jan 2023 15:57:51 +0000 Gerrit-HasComments: Yes