Todd Lipcon has submitted this change and it was merged.

Change subject: tools: wrap descriptions
......................................................................


tools: wrap descriptions

Wraps the help and usage output on a 78-column width so that it's much more
readable.

New output:

todd@todd-ThinkPad-T540p:~/git/kudu$ ./build/latest/bin/kudu cluster
Usage: ./build/latest/bin/kudu cluster <command> [<args>]

<command> can be one of the following:
  ksck   Check the health of a Kudu cluster. By default, ksck checks that
         master and tablet server processes are running, and that table
         metadata is consistent. Use the 'checksum' flag to check that tablet
         data is consistent (also see the 'tables' and 'tablets' flags). Use
         the 'checksum_snapshot' along with 'checksum' if the table or tablets
         are actively receiving inserts or updates.

todd@todd-ThinkPad-T540p:~/git/kudu$ ./build/latest/bin/kudu cluster ksck --help
Usage: ./build/latest/bin/kudu cluster ksck <master_address> [-checksum_scan]
        [-nochecksum_snapshot] [-tables=<tables>] [-tablets=<tablets>]
        [-color=<color>]

Check the health of a Kudu cluster. By default, ksck checks that master and
tablet server processes are running, and that table metadata is consistent.
Use the 'checksum' flag to check that tablet data is consistent (also see the
'tables' and 'tablets' flags). Use the 'checksum_snapshot' along with
'checksum' if the table or tablets are actively receiving inserts or updates.

    master_address (Kudu Master RPC address of form hostname:port)
      type: string default: ""
    -checksum_scan (Perform a checksum scan on data in the cluster.) type: bool
      default: false
    -checksum_snapshot (Should the checksum scanner use a snapshot scan)
      type: bool default: true
    -tables (Tables to check (comma-separated list of names). If not specified,
      checks all tables.) type: string default: ""
    -tablets (Tablets to check (comma-separated list of IDs) If not specified,
      checks all tablets.) type: string default: ""
    -color (Specifies whether ksck output is colorized. The default value
      'auto' colorizes output if the output is a terminal. The other valid
      values are 'always' or 'never'.) type: string default: "auto"

Change-Id: Ie63b72e1e1a3479819730c348f5a0f00a7164d02
Reviewed-on: http://gerrit.cloudera.org:8080/4130
Tested-by: Kudu Jenkins
Reviewed-by: Adar Dembo <a...@cloudera.com>
---
M src/kudu/tools/tool_action.cc
1 file changed, 58 insertions(+), 9 deletions(-)

Approvals:
  Adar Dembo: Looks good to me, approved
  Kudu Jenkins: Verified



-- 
To view, visit http://gerrit.cloudera.org:8080/4130
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie63b72e1e1a3479819730c348f5a0f00a7164d02
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <d...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>

Reply via email to