Fredy Wijaya has posted comments on this change. ( http://gerrit.cloudera.org:8080/13310 )
Change subject: IMPALA-8424: [DOCS] Support for column-level permissions on views ...................................................................... Patch Set 1: (4 comments) Do we have a doc somewhere that says column-level permissions/privileges on views aren't supported? If yes, I think we should update that doc by removing that info. If not, I guess the users can assume column-level permissions are supported on both tables and views. http://gerrit.cloudera.org:8080/#/c/13310/1/docs/topics/impala_grant.xml File docs/topics/impala_grant.xml: http://gerrit.cloudera.org:8080/#/c/13310/1/docs/topics/impala_grant.xml@62 PS1, Line 62: | VIEW This is incorrect, the syntax is still "TABLE" even for view. http://gerrit.cloudera.org:8080/#/c/13310/1/docs/topics/impala_grant.xml@71 PS1, Line 71: The <codeph><varname>object_type</varname></codeph> of <codeph>VIEW</codeph> is supported : for the <codeph>SELECT(<varname>column_name</varname>)</codeph> privilege. : </p> Remove this. This is incorrect. http://gerrit.cloudera.org:8080/#/c/13310/1/docs/topics/impala_revoke.xml File docs/topics/impala_revoke.xml: http://gerrit.cloudera.org:8080/#/c/13310/1/docs/topics/impala_revoke.xml@63 PS1, Line 63: | VIEW similarly, remove this. http://gerrit.cloudera.org:8080/#/c/13310/1/docs/topics/impala_revoke.xml@65 PS1, Line 65: <p> : The <codeph><varname>object_type</varname></codeph> of <codeph>VIEW</codeph> is supported : for the <codeph>SELECT(<varname>column_name</varname>)</codeph> privilege. : </p> : remove this as well. -- To view, visit http://gerrit.cloudera.org:8080/13310 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I7f95b37891b618b71460cd2ad444a422371455a6 Gerrit-Change-Number: 13310 Gerrit-PatchSet: 1 Gerrit-Owner: Alex Rodoni <[email protected]> Gerrit-Reviewer: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Comment-Date: Tue, 14 May 2019 21:00:30 +0000 Gerrit-HasComments: Yes
