Hello Quanlong Huang, Csaba Ringhofer, Wenzhe Zhou, Joe McDonnell, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/19179 to look at the new patch set (#7). Change subject: IMPALA-11669: (addendum) Set TConfiguration in TMemoryBuffer ...................................................................... IMPALA-11669: (addendum) Set TConfiguration in TMemoryBuffer This patch adds DefaultTConfiguration into TMemoryBuffer used within DeserializeThriftMsg, ThriftSerializer, and TSaslTransport. This patch also makes some adjustment, including: - Supply DefaultTConfiguration into the constructor of THttpTransport and TSaslTransport. - Supply DefaultTConfiguration through the constructor of TBufferedTransport. Testing: - Add DCHECK_EQ in places where we expect that it should pick up DefaultTConfiguration. - Add SerDeBuffer100MB test. - Pass core tests. - Manually run and pass test scenario described in testdata/scale_test_metadata/ both in SSL and no SSL setup. Change-Id: I37a8e71c64a09ec8aeccb96c6ee59ca82c0b37cb --- M be/src/rpc/TAcceptQueueServer.cpp M be/src/rpc/thrift-server.h M be/src/rpc/thrift-util-test.cc M be/src/rpc/thrift-util.cc M be/src/rpc/thrift-util.h A be/src/rpc/thrift-util.inline.h M be/src/transport/THttpTransport.cpp M be/src/transport/TSaslServerTransport.cpp M be/src/transport/TSaslTransport.cpp M be/src/util/parquet-reader.cc M testdata/scale_test_metadata/README.md 11 files changed, 146 insertions(+), 33 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/79/19179/7 -- To view, visit http://gerrit.cloudera.org:8080/19179 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I37a8e71c64a09ec8aeccb96c6ee59ca82c0b37cb Gerrit-Change-Number: 19179 Gerrit-PatchSet: 7 Gerrit-Owner: Riza Suminto <riza.sumi...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com> Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com> Gerrit-Reviewer: Wenzhe Zhou <wz...@cloudera.com>