impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ambreen Kazi (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] IMPALA-3401: [DOCS] Physically remove Cloudera Manager info
Date Thu, 09 Mar 2017 22:19:31 GMT
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 <jrussell@cloudera.com>
Gerrit-Reviewer: Ambreen Kazi <ambreen.kazi@cloudera.com>
Gerrit-HasComments: Yes

Mime
View raw message