Hello Will Berkeley, Kudu Jenkins, Grant Henke,

I'd like you to reexamine a change. Please visit

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

to look at the new patch set (#2).

Change subject: tools: escape brackets when generating XML
......................................................................

tools: escape brackets when generating XML

When building the site, we generate the XML for tools. Usually we escape
ambiguous characters for tooling arguments; it seems that wasn't the
case for arguments that are GFlags. This meant that before, we would run
into errors like the following when generating the site (new in 1.9.0
for the --predicates argument of the new `perf table_scan` tool):

/kudu/build/release/gen-docs/kudu.xml:49: parser error : StartTag: invalid 
element name
 * The 'Comparison' type supports <=, <, =, >, and >=,
                                       ^

Change-Id: I16c13d86b0b452e0559e245ee33373078e5e3713
---
M src/kudu/tools/tool_action.cc
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/81/12981/2
--
To view, visit http://gerrit.cloudera.org:8080/12981
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I16c13d86b0b452e0559e245ee33373078e5e3713
Gerrit-Change-Number: 12981
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com>

Reply via email to