John Russell has posted comments on this change. Change subject: IMPALA-4643: [DOCS] Rewrite issues.cloudera.org URLs ......................................................................
Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/6452/1/docs/impala_keydefs.ditamap File docs/impala_keydefs.ditamap: PS1, Line 316: <keydef href="https://issues.apache.org/jira/browse/IMPALA-1" scope="external" format="html" keys="IMPALA-1"/> > Unrelated to this review, but: This is wild! Is there a way to just have a No, XML really likes the generated output to be extremely predictable based on simple parsing of the input source files. Sometimes we employ postprocessing but (a) that's more complicated for an open source user to set up, and (b) the result would only apply to HTML output, whereas these links will work equally in HTML and PDF. http://gerrit.cloudera.org:8080/#/c/6452/1/docs/topics/impala_known_issues.xml File docs/topics/impala_known_issues.xml: PS1, Line 50: <xref href="https://issues.apache.org/" scope="external" format="html">issues.apache.org JIRA tracker</xref>. > If you look https://issues.apache.org/ is a page that gives you links to va Done -- To view, visit http://gerrit.cloudera.org:8080/6452 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I9ad2725e4afedddedf8ba65963b217803f532dea Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: John Russell <[email protected]> Gerrit-Reviewer: John Russell <[email protected]> Gerrit-Reviewer: Laurel Hale <[email protected]> Gerrit-Reviewer: Michael Brown <[email protected]> Gerrit-HasComments: Yes
