Hello Laszlo Gaal, Joe McDonnell, Michael Smith,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/24812
to look at the new patch set (#3).
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)
---
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(-)
git pull ssh://gerrit.cloudera.org:29418/native-toolchain
refs/changes/12/24812/3
--
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: newpatchset
Gerrit-Change-Id: I2228333e05d3f9009387406a96384098180c13d7
Gerrit-Change-Number: 24812
Gerrit-PatchSet: 3
Gerrit-Owner: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Laszlo Gaal <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
