Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/13235 )
Change subject: IMPALA-8364: [DOCS] Remove refereces to authz policy files ...................................................................... Patch Set 1: (4 comments) I think it's better if we don't mix the Ranger doc in this CR. http://gerrit.cloudera.org:8080/#/c/13235/1/docs/topics/impala_authorization.xml File docs/topics/impala_authorization.xml: http://gerrit.cloudera.org:8080/#/c/13235/1/docs/topics/impala_authorization.xml@106 PS1, Line 106: metastore database This is incorrect. Replace with "Stored inside the Sentry/Ranger database" http://gerrit.cloudera.org:8080/#/c/13235/1/docs/topics/impala_authorization.xml@110 PS1, Line 110: If you change privileges in Sentry or Ranger, e.g. adding a user, removing a user, : modifying privileges, you must clear the Impala Catalog server cache by running the This is a bit confusing. Maybe reword to something like if we you change privileges outside Impala, ... http://gerrit.cloudera.org:8080/#/c/13235/1/docs/topics/impala_authorization.xml@112 PS1, Line 112: NVALIDATE METADATA</codeph> statement. <codeph>INVALIDATE METADATA</codeph> is : not required if you make the changes to privileges within Impala. Replace INVALIDATE METADATA with REFRESH AUTHORIZATION instead since it's a more lightweight operation. http://gerrit.cloudera.org:8080/#/c/13235/1/docs/topics/impala_revoke.xml File docs/topics/impala_revoke.xml: http://gerrit.cloudera.org:8080/#/c/13235/1/docs/topics/impala_revoke.xml@114 PS1, Line 114: Ranger Ranger doesn't support roles. -- To view, visit http://gerrit.cloudera.org:8080/13235 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic85a74d81142803894d30c99cea0ec8a516bf756 Gerrit-Change-Number: 13235 Gerrit-PatchSet: 1 Gerrit-Owner: Alex Rodoni <[email protected]> Gerrit-Reviewer: Austin Nobis <[email protected]> Gerrit-Reviewer: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Comment-Date: Mon, 06 May 2019 17:14:16 +0000 Gerrit-HasComments: Yes
