Yifan Zhang has posted comments on this change. ( http://gerrit.cloudera.org:8080/18572 )
Change subject: [Tools] Rebuild master according to part of tables not all tables ...................................................................... Patch Set 31: Code-Review+1 (2 comments) Overall looks good now, just a few more nits. http://gerrit.cloudera.org:8080/#/c/18572/31/src/kudu/tools/master_rebuilder.cc File src/kudu/tools/master_rebuilder.cc: http://gerrit.cloudera.org:8080/#/c/18572/31/src/kudu/tools/master_rebuilder.cc@100 PS31, Line 100: tablet_by_name nit: maybe make it a default parameter with the default value 'nullptr', and then 'fake_map' will not be needed. http://gerrit.cloudera.org:8080/#/c/18572/31/src/kudu/tools/tool_action_master.cc File src/kudu/tools/tool_action_master.cc: http://gerrit.cloudera.org:8080/#/c/18572/31/src/kudu/tools/tool_action_master.cc@931 PS31, Line 931: " - The parameter 'tables' can be used to rebuild part of tables, when these tables " nit: I think we should highlight the difference between with and without '--tables' in the tool description. - Without '--tables', the tool will build a brand new sys.catalog table based on tablet data on tablet server metadata. - With '--tables', the tool will update specific tables in current sys.catalog. -- To view, visit http://gerrit.cloudera.org:8080/18572 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I45d812c8a16228ac937603872161969eb05136eb Gerrit-Change-Number: 18572 Gerrit-PatchSet: 31 Gerrit-Owner: Wang Xixu <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Wang Xixu <[email protected]> Gerrit-Reviewer: Yifan Zhang <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Comment-Date: Thu, 21 Jul 2022 09:21:45 +0000 Gerrit-HasComments: Yes
