Alex Rodoni 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)

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.
Done


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.
Done


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.
Done


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.
Done



--
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: Alex Rodoni <[email protected]>
Gerrit-Reviewer: Fredy Wijaya <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Wed, 15 May 2019 17:48:27 +0000
Gerrit-HasComments: Yes

Reply via email to