Lars Volker 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 3: Code-Review+1

(1 comment)

LGTM but someone with more expertise in Authorization should look at it, too

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@934
PS3, Line 934:           <codeblock>data_read = 
server=server1-&gt;uri=file:///path/to/dir,\ 
server=server1-&gt;uri=hdfs://ha-nn-uri/path/to/dir
nit: here, too?



--
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: 3
Gerrit-Owner: Alex Rodoni <[email protected]>
Gerrit-Reviewer: Alex Rodoni <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Lars Volker <[email protected]>
Gerrit-Comment-Date: Wed, 06 Jun 2018 18:08:34 +0000
Gerrit-HasComments: Yes

Reply via email to