Ambreen Kazi has posted comments on this change. Change subject: IMPALA-3401: [DOCS] Physically remove Cloudera Manager info ......................................................................
Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/6325/2/docs/topics/impala_logging.xml File docs/topics/impala_logging.xml: PS2, Line 372: Cloudera Cloudera ref - there's at least one more on this page. PS2, Line 453: <p> : See : <xref audience="integrated" href="sg_redaction.xml#log_redact"/><xref audience="standalone" href="http://www.cloudera.com/documentation/enterprise/latest/topics/sg_redaction.html"; scope="external" format="html"/> : for details about how to enable this feature and set : up the regular expressions to detect and redact sensitive information within SQL statement text. : </p> This should've been set to hidden and removed -- the content at that link is very strictly geared towards CM. We need clarification on how redaction works with standalone Impala on hadoop. How it can be enabled and what the default setting is. http://gerrit.cloudera.org:8080/#/c/6325/2/docs/topics/impala_txtfile.xml File docs/topics/impala_txtfile.xml: PS2, Line 489: Cloudera will this continue to be hosted on archive.cloudera? Flagging this because Cloudera was removed from the previous paragraph that talks about this package, but not here. http://gerrit.cloudera.org:8080/#/c/6325/2/docs/topics/impala_udf.xml File docs/topics/impala_udf.xml: PS2, Line 386: /cloudera/ cloudera repo link to be replaced? -- To view, visit http://gerrit.cloudera.org:8080/6325 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ifc4a84527ae42c39b3717190b6cf669e17fff04b Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: John Russell <[email protected]> Gerrit-Reviewer: Ambreen Kazi <[email protected]> Gerrit-HasComments: Yes
