Yifan Zhang has posted comments on this change. ( http://gerrit.cloudera.org:8080/19411 )
Change subject: [tools] add --tables flag to 'local_replica delete' ...................................................................... Patch Set 6: (4 comments) http://gerrit.cloudera.org:8080/#/c/19411/5//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/19411/5//COMMIT_MSG@16 PS5, Line 16: cannot > Do you mean "cannot" ? Done http://gerrit.cloudera.org:8080/#/c/19411/5//COMMIT_MSG@16 PS5, Line 16: support > nit: supports Done http://gerrit.cloudera.org:8080/#/c/19411/5//COMMIT_MSG@17 PS5, Line 17: whether they specified invalid tablet id patterns, the non-exist : tablet id patterns will be ignored even though --ignore_nonexistent : > Do you mean tablet id patterns will be treated as invalid? I mean that non-exist tablet id patterns will be ignored even though the -ignore_nonexistent flag is false. But the non-exist tablet ids will not. http://gerrit.cloudera.org:8080/#/c/19411/5/src/kudu/tools/kudu-tool-test.cc File src/kudu/tools/kudu-tool-test.cc: http://gerrit.cloudera.org:8080/#/c/19411/5/src/kudu/tools/kudu-tool-test.cc@4320 PS5, Line 4320: --tables=$1 > If multiple tables can be entered by user, could you add another test case Done -- 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: 6 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: Tue, 17 Jan 2023 03:58:19 +0000 Gerrit-HasComments: Yes