Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/21273 )
Change subject: KUDU-3326 correct error messages in tool usage instructions ...................................................................... KUDU-3326 correct error messages in tool usage instructions This patch mainly fixes the error messages in the recall tool as follows: ` Usage: kudu table recall <master_addresses> <tablet_id> [-new_table_name=<name>] ` We expect to recall tables in soft delete state using the table ID, not the tablet ID. The correct output is as follows: ` Usage: kudu table recall <master_addresses> <table_id> [-new_table_name=<name>] ` This is just about correcting the errors in the prompts and does not involve any related recall logic. Therefore, there are no additional unit tests added. Change-Id: Ib1c6df0806eee78280b00f6a528d42d434a63e2f Reviewed-on: http://gerrit.cloudera.org:8080/21266 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Alexey Serbin <[email protected]> (cherry picked from commit d06f35db2e5a0e93a3dcab0d3b26e4443914df89) Reviewed-on: http://gerrit.cloudera.org:8080/21273 Reviewed-by: Yingchun Lai <[email protected]> --- M src/kudu/tools/tool_action_common.cc M src/kudu/tools/tool_action_common.h M src/kudu/tools/tool_action_table.cc 3 files changed, 4 insertions(+), 2 deletions(-) Approvals: Yingchun Lai: Looks good to me, approved Alexey Serbin: Verified -- To view, visit http://gerrit.cloudera.org:8080/21273 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.17.x Gerrit-MessageType: merged Gerrit-Change-Id: Ib1c6df0806eee78280b00f6a528d42d434a63e2f Gerrit-Change-Number: 21273 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: KeDeng <[email protected]> Gerrit-Reviewer: Yifan Zhang <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]>
