Csaba Ringhofer has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/24812 )
Change subject: IMPALA-15343: Add thrift 0.24.0 ...................................................................... IMPALA-15343: Add thrift 0.24.0 Build thrift as 0.24.0-p3, carrying forward only the 0.16.0-p7 patches that are still needed: - 0001 Fix Thread destructor sigsegv when uninitialized - 0002 THRIFT-5705: Don't accumulate retry count for TSSLSocket read() - 0003 IMPALA-13020: Use 64-bit integer for max message size Dropped from the 0.16.0 set: - THRIFT-5716 (TMemoryBuffer uint32 overflow) and THRIFT-5670 (GlobalOutput.perror misuse): already fixed in thrift 0.24.0. - TLS forward-compatibility and IMPALA-11195 (disable SSL renegotiation): TLS is to be configured from Impala by injecting a pre-built SSL_CTX (made possible by THRIFT-3165 in Thrift 0.24.0) rather than by patching Thrift. https://gerrit.cloudera.org/#/c/24814/ has the Impala side changes to switch to Thrift 0.24.0-p3 in c++, including SSL_CTX initialization. Change-Id: I2228333e05d3f9009387406a96384098180c13d7 Assisted-By: Claude Opus 4.8 (1M context) Reviewed-on: http://gerrit.cloudera.org:8080/24812 Reviewed-by: Michael Smith <[email protected]> Reviewed-by: Laszlo Gaal <[email protected]> Tested-by: Csaba Ringhofer <[email protected]> --- M buildall.sh A source/thrift/thrift-0.24.0-patches/0001-Fix-Thread-destructor-sigsegv-when-uninitialized.patch A source/thrift/thrift-0.24.0-patches/0002-THRIFT-5705-Don-t-accumulate-accumulate-retry-count-.patch A source/thrift/thrift-0.24.0-patches/0003-IMPALA-13020-Use-64-bit-integer-for-max-message-size.patch 4 files changed, 231 insertions(+), 0 deletions(-) Approvals: Michael Smith: Looks good to me, but someone else must approve Laszlo Gaal: Looks good to me, approved Csaba Ringhofer: Verified -- To view, visit http://gerrit.cloudera.org:8080/24812 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: native-toolchain Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I2228333e05d3f9009387406a96384098180c13d7 Gerrit-Change-Number: 24812 Gerrit-PatchSet: 5 Gerrit-Owner: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Laszlo Gaal <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]>
