Ambreen Kazi has posted comments on this change. Change subject: IMPALA-3410 [DOCS] Rework Impala authentication topics to be generic ......................................................................
Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/5962/2/docs/topics/impala_kerberos.xml File docs/topics/impala_kerberos.xml: PS2, Line 46: KDC > Not defined yet Added the full form with a link to relevant MIT KDC page. Line 50: <p audience="hidden"> > Can this be removed? Did you mean the entire paragraph that talks about Impala in a CM env? I've tagged it 'hidden' for now to be consistent with the rest of John's updates. We can deal with these occurrences all at once then. http://gerrit.cloudera.org:8080/#/c/5962/2/docs/topics/impala_ldap.xml File docs/topics/impala_ldap.xml: Line 189: Specify the option on the <cmdname>impalad</cmdname> command line. > This paragraph is confusing now. Removed this line. Added this detail to a line before the ldap bind options are listed. -- To view, visit http://gerrit.cloudera.org:8080/5962 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I639a55eb43555cf074c26d23b5c72f778073231c Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Ambreen Kazi <ambreen.k...@cloudera.com> Gerrit-Reviewer: Ambreen Kazi <ambreen.k...@cloudera.com> Gerrit-Reviewer: Jim Apple <jbapple-imp...@apache.org> Gerrit-Reviewer: John Russell <jruss...@cloudera.com> Gerrit-Reviewer: Laurel Hale <lau...@cloudera.com> Gerrit-HasComments: Yes