Hello Pooja Nilangekar, Joe McDonnell, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/11625
to look at the new patch set (#2).
Change subject: IMPALA-7678: Reapply "IMPALA-7660: Support ECDH ciphers for
debug webserver"
......................................................................
IMPALA-7678: Reapply "IMPALA-7660: Support ECDH ciphers for debug webserver"
This patch reverses the revert of IMPALA-7660.
The problem with IMPALA-7660 was that urllib.urlopen added the
'context' parameter in 2.7.9, so it isn't present on rhel7, which uses
2.7.5
urllib.urlopen has been deprecated since 2.6 anyways, so the fix is to
switch to using the more current urllib2.
This patch also adds more info to the error message printed by
start-impala-cluster.py when the debug webserver cannot be reached yet
to help with debugging these issues in the future.
Testing:
- Ran a full build on rhel7
Change-Id: I679469ed7f27944f75004ec4b16d513e6ea6b544
---
M be/src/thirdparty/squeasel/squeasel.c
M tests/common/impala_cluster.py
M tests/common/impala_service.py
M tests/custom_cluster/test_client_ssl.py
M tests/custom_cluster/test_redaction.py
M tests/run-tests.py
6 files changed, 106 insertions(+), 46 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/25/11625/2
--
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: newpatchset
Gerrit-Change-Id: I679469ed7f27944f75004ec4b16d513e6ea6b544
Gerrit-Change-Number: 11625
Gerrit-PatchSet: 2
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>
