John Russell has posted comments on this change.

Change subject: [DOCS] Mention Kerberos and TLS for Kudu security
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/6634/1/docs/shared/impala_common.xml
File docs/shared/impala_common.xml:

PS1, Line 3380: kudu_common
> I don't see how this is related?
It's purely a cosmetic change for the convenience of doc maintainers. The 
former id="kudu" was the same as in impala_kudu.xml which sometimes caused text 
editors to load the wrong file.


-- 
To view, visit http://gerrit.cloudera.org:8080/6634
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1266ad38468ef2e987aff5aaaa4db35e6cafdacc
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell <jruss...@cloudera.com>
Gerrit-Reviewer: Ambreen Kazi <ambreen.k...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org>
Gerrit-Reviewer: John Russell <jruss...@cloudera.com>
Gerrit-Reviewer: Matthew Jacobs <m...@cloudera.com>
Gerrit-HasComments: Yes

Reply via email to