Ambreen Kazi has posted comments on this change. Change subject: IMPALA-3410 [DOCS] Rework Impala security topics to be generic ......................................................................
Patch Set 1: (14 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) Done http://gerrit.cloudera.org:8080/#/c/5931/1/docs/shared/impala_common.xml File docs/shared/impala_common.xml: PS1, Line 608: audience="Cloudera" > This should be audience="hidden". (There shouldn't be any audience="Clouder I found it like this. Changed to hidden now. PS1, Line 616: appropropriate > note to self - typo Done http://gerrit.cloudera.org:8080/#/c/5931/1/docs/topics/impala_authorization.xml File docs/topics/impala_authorization.xml: PS1, Line 319: recommends : managing privileges through SQL statements, as described in : <xref href="impala_authorization.xml#sentry_service"/>. If you are still using policy files, plan to : migrate to the new approach some time in the future. > Should we genericize the wording but keep the recommendation to use policy Genericized the wording. The recommendation should be to use the SQL statements. I've removed the 'some time in the future' wording as well because it's been a while since the Sentry service released. 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 Done PS1, Line 853: </p> > For example: Done PS1, Line 951: we strongly recommend > I've stayed away from "we recommend" because there isn't the same commitmen I've reworded this note to make it more of an instruction rather than a suggestion/recommendation. 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". Done PS1, Line 45: focussed > focused (American spelling) Done PS1, Line 116: > hyphen, like previous instance Done http://gerrit.cloudera.org:8080/#/c/5931/1/docs/topics/impala_security_install.xml File docs/topics/impala_security_install.xml: PS1, Line 38: > Trailing blank. Done http://gerrit.cloudera.org:8080/#/c/5931/1/docs/topics/impala_ssl.xml File docs/topics/impala_ssl.xml: PS1, Line 39: > Let's start the text on the line after the <indexterm> tag. Done PS1, Line 44: <!-- Formerly: conref="../shared/CDHVariables.xml#xd_583c10bfdbd326ba-3ca24a24-13d801432 > We can get get of this comment. Done PS1, Line 240: > Trailing blank. Done -- 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