Wang Xixu has posted comments on this change. ( http://gerrit.cloudera.org:8080/19501 )
Change subject: [Tool] Show the information of a tablet ...................................................................... Patch Set 9: (4 comments) > Patch Set 8: > > (1 comment) http://gerrit.cloudera.org:8080/#/c/19501/7//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/19501/7//COMMIT_MSG@17 PS7, Line 17: follow > following Done http://gerrit.cloudera.org:8080/#/c/19501/7/src/kudu/master/catalog_manager.cc File src/kudu/master/catalog_manager.cc: http://gerrit.cloudera.org:8080/#/c/19501/7/src/kudu/master/catalog_manager.cc@6276 PS7, Line 6276: > nit: you could use std::move() here to prevent copying: Done http://gerrit.cloudera.org:8080/#/c/19501/7/src/kudu/tools/tool_action_tablet.cc File src/kudu/tools/tool_action_tablet.cc: http://gerrit.cloudera.org:8080/#/c/19501/7/src/kudu/tools/tool_action_tablet.cc@126 PS7, Line 126: Tabl > nit: Tablet Server UUID Done http://gerrit.cloudera.org:8080/#/c/19501/7/src/kudu/tools/tool_action_tablet.cc@414 PS7, Line 414: Show information on the table which the tablet is a > nit: maybe change this to be more specific Done. Thanks for your comments. -- To view, visit http://gerrit.cloudera.org:8080/19501 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib5ae5f61f50a44c4787843df76adaa61700ae9fe Gerrit-Change-Number: 19501 Gerrit-PatchSet: 9 Gerrit-Owner: Wang Xixu <1450306...@qq.com> Gerrit-Reviewer: Alexey Serbin <ale...@apache.org> Gerrit-Reviewer: KeDeng <kdeng...@gmail.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Wang Xixu <1450306...@qq.com> Gerrit-Reviewer: Yingchun Lai <laiyingc...@apache.org> Gerrit-Reviewer: Yuqi Du <shenxingwuy...@gmail.com> Gerrit-Comment-Date: Mon, 20 Mar 2023 03:19:06 +0000 Gerrit-HasComments: Yes