Fredy Wijaya has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10551 )

Change subject: IMPALA-5380: [DOCS] Added additional filesystems supported in 
URI
......................................................................


Patch Set 4:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/10551/3/docs/topics/impala_authorization.xml
File docs/topics/impala_authorization.xml:

http://gerrit.cloudera.org:8080/#/c/10551/3/docs/topics/impala_authorization.xml@921
PS3, Line 921: If a URI starts with anything else, it will
             :         cause an exception
I don't think this is correct. If a URI starts with an absolute path, it will 
automatically append the default prefix filesystem. For example:

> grant all on uri '/tmp' to role test_role;
> show grant role test_role;
Query: show grant role test_role
+-------+----------+-------+--------+------------------------------------------------------+-----------+--------------+-------------------------------+
| scope | database | table | column | uri                                       
           | privilege | grant_option | create_time                   |
+-------+----------+-------+--------+------------------------------------------------------+-----------+--------------+-------------------------------+
| uri   |          |       |        | hdfs://localhost:20500/tmp                
           | all       | false        | Wed, Jun 06 2018 11:22:26.283 |



--
To view, visit http://gerrit.cloudera.org:8080/10551
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I11d57efd564fb5779539a88d7b209cee93d313c6
Gerrit-Change-Number: 10551
Gerrit-PatchSet: 4
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-Reviewer: Lars Volker <[email protected]>
Gerrit-Comment-Date: Wed, 06 Jun 2018 18:24:39 +0000
Gerrit-HasComments: Yes

Reply via email to