Jim Apple has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11853 )

Change subject: IMPALA-7244: [DOCS] Impala supports ADLS Gen 2 (ABFS)
......................................................................


Patch Set 2:

(1 comment)

Thanks for taking this on, Alex! I also agree with Joe's comments on PS1.

http://gerrit.cloudera.org:8080/#/c/11853/2/docs/topics/impala_adls.xml
File docs/topics/impala_adls.xml:

http://gerrit.cloudera.org:8080/#/c/11853/2/docs/topics/impala_adls.xml@92
PS2, Line 92:               <codeph>abfs(s)://</codeph> for ADLS Gen 2, for the
nit: let's expand it out "abfs:// or abfss://".



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic06d9ac92ed78b9092369e211de8a81db1d7ce90
Gerrit-Change-Number: 11853
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Rodoni <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Jim Apple <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Comment-Date: Mon, 05 Nov 2018 21:44:20 +0000
Gerrit-HasComments: Yes

Reply via email to