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