Hello Tim Armstrong, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16704
to look at the new patch set (#2).
Change subject: IMPALA-6861: Fix OpenSSL initialization
......................................................................
IMPALA-6861: Fix OpenSSL initialization
Impalads currently initialize OpenSSL twice: once when initializing
Thrift and once when initializing KRPC. The initialization is
theoretically idempotent but not thread-safe, so its better to clean
this up.
This patch disables the Thrift version as its older (last updated in
2015) and the KRPC version contains logic specific to more recent
versions of OpenSSL. The catalogd and statestored also now use the
KRPC version instead of the Thrift version.
It also improves debuggability by adding some additional startup
logging.
Testing:
- Passed existing SSL tests.
Change-Id: I35b1362d40c8a12082cc8b531a38b4a485bac0e7
---
M be/src/rpc/authentication.cc
M be/src/rpc/authentication.h
M be/src/rpc/thrift-server.cc
3 files changed, 11 insertions(+), 10 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/04/16704/2
--
To view, visit http://gerrit.cloudera.org:8080/16704
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I35b1362d40c8a12082cc8b531a38b4a485bac0e7
Gerrit-Change-Number: 16704
Gerrit-PatchSet: 2
Gerrit-Owner: Thomas Tauber-Marshall <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
