Tamas Mate has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13960 )

Change subject: IMPALA-8456: [DOCS] New HTTP protocol for Impala clients
......................................................................


Patch Set 1:

(1 comment)

Hi Alex,
thank you for the doc.

http://gerrit.cloudera.org:8080/#/c/13960/1/docs/topics/impala_client.xml
File docs/topics/impala_client.xml:

http://gerrit.cloudera.org:8080/#/c/13960/1/docs/topics/impala_client.xml@208
PS1, Line 208: HTTP(S)
Small nit, this is a bit ambiguous for me. I believe it could be something like:

To enable SSL for HiveServer2 HTTP endpoint use --ssl_server_certificate and 
--ssl_private_key.

The 'impala_ssl.xml' could be linked as well for reference.

What do you think?



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3101f8babc77a5a872778499a54ac479a66ad996
Gerrit-Change-Number: 13960
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Rodoni <arod...@cloudera.com>
Gerrit-Reviewer: Bharath Vissapragada <bhara...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Michael Ho <k...@cloudera.com>
Gerrit-Reviewer: Tamas Mate <tm...@cloudera.com>
Gerrit-Reviewer: Thomas Tauber-Marshall <tmarsh...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-Comment-Date: Wed, 31 Jul 2019 16:47:37 +0000
Gerrit-HasComments: Yes

Reply via email to