Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16704 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/16704 Reviewed-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- 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(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- 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: merged Gerrit-Change-Id: I35b1362d40c8a12082cc8b531a38b4a485bac0e7 Gerrit-Change-Number: 16704 Gerrit-PatchSet: 4 Gerrit-Owner: Thomas Tauber-Marshall <tmarsh...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Thomas Tauber-Marshall <tmarsh...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
