Zoltan Chovan has posted comments on this change. ( http://gerrit.cloudera.org:8080/21496 )
Change subject: Include table UUID when using --show_table_info ...................................................................... Patch Set 3: (1 comment) No, I forgot to update the tests, fixed it now. Also I changed the order and made the new ID field the last, so existing scripts are less likely to fail due to this change. http://gerrit.cloudera.org:8080/#/c/21496/1/src/kudu/tools/tool_action_table.cc File src/kudu/tools/tool_action_table.cc: http://gerrit.cloudera.org:8080/#/c/21496/1/src/kudu/tools/tool_action_table.cc@123 PS1, Line 123: DEFINE_bool(show_table_info, false, : "Include extra information such as table id, number of tablets, " : "replicas, and live row count for a table in the > Can you please extend this description to include the newly added table id? Done -- To view, visit http://gerrit.cloudera.org:8080/21496 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I324b920e6feb6139e7d884e3cf08069b0cb922a4 Gerrit-Change-Number: 21496 Gerrit-PatchSet: 3 Gerrit-Owner: Zoltan Chovan <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Reviewer: Zoltan Chovan <[email protected]> Gerrit-Reviewer: Zoltan Martonka <[email protected]> Gerrit-Comment-Date: Tue, 11 Jun 2024 14:41:35 +0000 Gerrit-HasComments: Yes
