Jason Fehr has uploaded a new patch set (#3). ( 
http://gerrit.cloudera.org:8080/19428 )

Change subject: IMPALA-11850 Adds HTTP tracing headers when using the hs2-http 
protocol.
......................................................................

IMPALA-11850 Adds HTTP tracing headers when using the hs2-http protocol.

When using the hs2 protocol with the http transport, include several tracing 
http
headers by default.  These headers are:

  * X-Request-Id        -- client defined string that identifies the http
                           request, this string is meaningful only to the client
  * X-Impala-Session-Id -- session id generated by the Impala backend, will be
                           omitted on http calls that occur before this id has
                           been generated
  * X-Impala-Query-Id   -- query id generated by the Impala backend, will be
                           omitted on http calls that occur before this id has
                           been generated

The Impala shell includes these flags by default.  Command line
arguments have been added to remove these headers.

The Impala backend logs out these headers if they are on the http
request.

Testing:
  - manual testing (verified using debugging proxy and impala logs)
  - new python test

Change-Id: I7857eb5ec03eba32e06ec8d4133480f2e958ad2f
---
M be/src/transport/THttpServer.cpp
M be/src/transport/THttpServer.h
M shell/ImpalaHttpClient.py
M shell/impala_client.py
M shell/impala_shell.py
M shell/impala_shell_config_defaults.py
M shell/option_parser.py
M tests/common/test_dimensions.py
M tests/shell/test_shell_commandline.py
9 files changed, 241 insertions(+), 10 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/28/19428/3
--
To view, visit http://gerrit.cloudera.org:8080/19428
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I7857eb5ec03eba32e06ec8d4133480f2e958ad2f
Gerrit-Change-Number: 19428
Gerrit-PatchSet: 3
Gerrit-Owner: Jason Fehr <jf...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>

Reply via email to