Hello Dan Burkert, Adar Dembo,

I'd like you to do a code review.  Please visit

    http://gerrit.cloudera.org:8080/4130

to review the following change.

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
---
M src/kudu/tools/tool_action.cc
1 file changed, 58 insertions(+), 9 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/30/4130/1
-- 
To view, visit http://gerrit.cloudera.org:8080/4130
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie63b72e1e1a3479819730c348f5a0f00a7164d02
Gerrit-PatchSet: 1
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>

Reply via email to