You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Henry Robinson (Code Review)" <ge...@cloudera.org> on 2016/06/13 23:19:32 UTC

[Impala-CR](cdh5-trunk) IMPALA-3736: Move Impala HTTP handlers to a separate class

Hello Matthew Jacobs,

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

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

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

Change subject: IMPALA-3736: Move Impala HTTP handlers to a separate class
......................................................................

IMPALA-3736: Move Impala HTTP handlers to a separate class

HTTP handler callbacks take a lot of header room, partly because they
usually have their JSON output documented literally. This patch moves
ImpalaServer's HTTP handlers out of ImpalaServer itself, into a friend
class.

Change-Id: I8453b3367653914163ca6acae48e7605f73cc675
---
M be/src/service/CMakeLists.txt
R be/src/service/impala-http-handler.cc
A be/src/service/impala-http-handler.h
M be/src/service/impala-server.cc
M be/src/service/impala-server.h
5 files changed, 269 insertions(+), 222 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/70/3370/2
-- 
To view, visit http://gerrit.cloudera.org:8080/3370
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I8453b3367653914163ca6acae48e7605f73cc675
Gerrit-PatchSet: 2
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Henry Robinson <he...@cloudera.com>
Gerrit-Reviewer: Henry Robinson <he...@cloudera.com>
Gerrit-Reviewer: Matthew Jacobs <mj...@cloudera.com>