Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23660 )
Change subject: [tools] update 'kudu master add' in-line help ...................................................................... [tools] update 'kudu master add' in-line help Since the 'kudu master add' CLI tool has become de facto obsolete after a significant improvement in usability of adding new Kudu masters in the 1.16.0 release [1], it makes sense to reflect on that in the inline tool's help/usage blurb. This changelist does exactly so by adding an appropriate NOTE. In addition, I updated the help message to clarify on a few crucial points and also removed irrelevant optional flag --logtostderr from the list of the flags advertised by the tool. [1] https://github.com/apache/kudu/commit/7e66534d0 Change-Id: Ida92fda5d6d3f4edd9f04796efa090bef7c540bd Reviewed-on: http://gerrit.cloudera.org:8080/23647 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Abhishek Chennaka <[email protected]> (cherry picked from commit 706af2d0bc1ba6845527b967ef777c208707be20) Reviewed-on: http://gerrit.cloudera.org:8080/23660 --- M src/kudu/tools/kudu-tool-test.cc M src/kudu/tools/tool_action_master.cc 2 files changed, 24 insertions(+), 23 deletions(-) Approvals: Abhishek Chennaka: Looks good to me, approved Alexey Serbin: Verified -- To view, visit http://gerrit.cloudera.org:8080/23660 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.18.x Gerrit-MessageType: merged Gerrit-Change-Id: Ida92fda5d6d3f4edd9f04796efa090bef7c540bd Gerrit-Change-Number: 23660 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
