Andrew Sherman has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15620 )
Change subject: Patch Thrift to 0.9.3-p8 to remove a duplicate 'Host' header in http messages. ...................................................................... Patch Thrift to 0.9.3-p8 to remove a duplicate 'Host' header in http messages. This will be used to fix IMPALA-9540 "Impala Shell sends duplicate "Host" headers in http mode". Many http servers will not accept an http request that has multiple copies of the Host header. Remove the extraneous call which adds an unnecessary header. TESTING: I built the toolchain using jenkins. I built impala with the new toolchain, ran all Impala end-to-end tests, and tested that IMPALA-9540 is fixed. Change-Id: If5fbeeba4c55423128733ec15a3dbabf2b6ac8a6 Reviewed-on: http://gerrit.cloudera.org:8080/15620 Reviewed-by: Tim Armstrong <[email protected]> Tested-by: Andrew Sherman <[email protected]> --- M buildall.sh A source/thrift/thrift-0.9.3-patches/0008-remove-duplicated-host-header.patch 2 files changed, 28 insertions(+), 1 deletion(-) Approvals: Tim Armstrong: Looks good to me, approved Andrew Sherman: Verified -- To view, visit http://gerrit.cloudera.org:8080/15620 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: native-toolchain Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: If5fbeeba4c55423128733ec15a3dbabf2b6ac8a6 Gerrit-Change-Number: 15620 Gerrit-PatchSet: 2 Gerrit-Owner: Andrew Sherman <[email protected]> Gerrit-Reviewer: Andrew Sherman <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
