Yingchun Lai has posted comments on this change. ( http://gerrit.cloudera.org:8080/19501 )
Change subject: [Tool] Show the information of a tablet ...................................................................... Patch Set 11: (2 comments) http://gerrit.cloudera.org:8080/#/c/19501/11/src/kudu/master/catalog_manager.h File src/kudu/master/catalog_manager.h: http://gerrit.cloudera.org:8080/#/c/19501/11/src/kudu/master/catalog_manager.h@802 PS11, Line 802: Remove one blank line. http://gerrit.cloudera.org:8080/#/c/19501/11/src/kudu/master/master.proto File src/kudu/master/master.proto: http://gerrit.cloudera.org:8080/#/c/19501/11/src/kudu/master/master.proto@459 PS11, Line 459: message TableInfoPB { Of course it's convenient to obtain table info by sending GetTabletLocations request, however, there is a 'table info' in ListTablesResponsePB, why not reuse it? If some day we want some more 'table info' by GetTabletLocations request, will add more fields in TabletLocationsPB.TableInfoPB? Is reuse ListTablesResponsePB, the new CLI tool is compatiable to old server too. -- 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: 11 Gerrit-Owner: Wang Xixu <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: KeDeng <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Wang Xixu <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Reviewer: Yuqi Du <[email protected]> Gerrit-Comment-Date: Fri, 31 Mar 2023 09:12:24 +0000 Gerrit-HasComments: Yes
