Alexey Serbin has posted comments on this change.

Change subject: KUDU-1708. Document Kudu command-line tools
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/6645/2/src/kudu/tools/tool_action.cc
File src/kudu/tools/tool_action.cc:

Line 190:   return xml;
> It's possible, but I followed the pattern used by existing helpxml flags. I
I see.

I just thought if it were possible to produce output (not only help) using 
libxo, it might be useful in troubleshooting and, maybe, even creating some 
automated scripts (the JSON output might be the preferred one in those 
scenarios).

If it's just for documenting the help output which is going to be processed by 
the Kudu documentation suite, then HTML or JSON output is not needed as is.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9f484f772cbaeb385687d83a2665ae4d7292aaf5
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: branch-1.3.x
Gerrit-Owner: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Grant Henke <granthe...@gmail.com>
Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-HasComments: Yes

Reply via email to