Jim Apple has posted comments on this change. Change subject: IMPALA-3410 [DOCS] Rework Impala security topics to be generic ......................................................................
Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/5931/1//COMMIT_MSG Commit Message: Line 10: topics. References to CDH and Cloudera Manager docs/products have been either wrap at 70 characters (the red line) http://gerrit.cloudera.org:8080/#/c/5931/1/docs/topics/impala_authorization.xml File docs/topics/impala_authorization.xml: Line 850: To enable URIs in per-DB policy files, the Java configuration option <codeph>sentry.allow.uri.db.policyfile</codeph> must be set to <codeph>true</codeph>. Shorter lines maker for easier reviews http://gerrit.cloudera.org:8080/#/c/5931/1/docs/topics/impala_security.xml File docs/topics/impala_security.xml: PS1, Line 40: the Apache Sentry : open source project Optional: can be shortened to just "Apache Sentry". -- To view, visit http://gerrit.cloudera.org:8080/5931 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ie5c4431f3236b18fc282343ed98513f0e578130e Gerrit-PatchSet: 1 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