Thomas Marshall has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11625 )

Change subject: IMPALA-7678: Reapply "IMPALA-7660: Support ECDH ciphers for 
debug webserver"
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/11625/1/tests/common/impala_service.py
File tests/common/impala_service.py:

http://gerrit.cloudera.org:8080/#/c/11625/1/tests/common/impala_service.py@62
PS1, Line 62: context=context
> One option is to use python for our non-SSL stuff and call out to wget or a
I think I'll just switch to using the 'requests' library, which is in 
infra/python/deps/requirements.txt so it shouldn't suffer from these version 
issues.

I've run into some additional problems that need to be worked through though.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I679469ed7f27944f75004ec4b16d513e6ea6b544
Gerrit-Change-Number: 11625
Gerrit-PatchSet: 1
Gerrit-Owner: Thomas Marshall <thomasmarsh...@cmu.edu>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>
Gerrit-Reviewer: Pooja Nilangekar <pooja.nilange...@cloudera.com>
Gerrit-Reviewer: Thomas Marshall <thomasmarsh...@cmu.edu>
Gerrit-Comment-Date: Wed, 10 Oct 2018 18:37:03 +0000
Gerrit-HasComments: Yes

Reply via email to