[Impala-ASF-CR] IMPALA-9414 (part 2): Support the 'Expect: 100-continue' http header

["Thomas Tauber-Marshall (Code Review)" <ge...@cloudera.org>]

Hello David Knupp, Impala Public Jenkins, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/15284

to look at the new patch set (#4).

Change subject: IMPALA-9414 (part 2): Support the 'Expect: 100-continue' http header
......................................................................

IMPALA-9414 (part 2): Support the 'Expect: 100-continue' http header

The 'Expect: 100-continue' http header allows http clients to send
only the headers for their request, get a confirmation back from the
server that the headers are valid, and only then send the body of the
request, avoiding the overhead of sending large requests that will
ultimately fail.

This patch adds support for this in the HS2 HTTP server by having
THttpServer look for the header, and if it's present and the request
is validated returning a '100 Continue' response before reading the
body of the request.

It also adds supports for using this header on large requests sent by
impala-shell.

Testing:
- This case is covered by the existing test_large_sql, however that
  test was previously broken and passing spuriously. This patch fixes
  the test.

Change-Id: I4153968551acd58b25c7923c2ebf75ee29a7e76b
---
M be/src/transport/THttpServer.cpp
M be/src/transport/THttpTransport.cpp
M be/src/transport/THttpTransport.h
M shell/THttpClient.py
M shell/impala_client.py
M shell/impala_shell.py
M shell/make_shell_tarball.sh
M shell/packaging/make_python_package.sh
A shell/shell_exceptions.py
M tests/shell/test_shell_commandline.py
10 files changed, 99 insertions(+), 52 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/84/15284/4
-- 
To view, visit http://gerrit.cloudera.org:8080/15284
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I4153968551acd58b25c7923c2ebf75ee29a7e76b
Gerrit-Change-Number: 15284
Gerrit-PatchSet: 4
Gerrit-Owner: Thomas Tauber-Marshall <tm...@cloudera.com>
Gerrit-Reviewer: David Knupp <dk...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Thomas Tauber-Marshall <tm...@cloudera.com>