Lars Volker has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10767 )

Change subject: Update Impala to adapt to KRPC changes
......................................................................


Patch Set 2:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/10767/2/be/src/rpc/impala-service-pool.cc
File be/src/rpc/impala-service-pool.cc:

http://gerrit.cloudera.org:8080/#/c/10767/2/be/src/rpc/impala-service-pool.cc@223
PS2, Line 223:     incoming->RecordHandlingStarted(incoming_queue_time_.get());
This API changed.


http://gerrit.cloudera.org:8080/#/c/10767/2/be/src/runtime/krpc-data-stream-mgr.cc
File be/src/runtime/krpc-data-stream-mgr.cc:

http://gerrit.cloudera.org:8080/#/c/10767/2/be/src/runtime/krpc-data-stream-mgr.cc@26
PS2, Line 26: #include "kudu/util/net/sockaddr.h"
After some cleanup on KRPC's side this include became necessary.



--
To view, visit http://gerrit.cloudera.org:8080/10767
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5424b54fbb6ab727c9daeaeea3032384fe6c53be
Gerrit-Change-Number: 10767
Gerrit-PatchSet: 2
Gerrit-Owner: Lars Volker <[email protected]>
Gerrit-Reviewer: Lars Volker <[email protected]>
Gerrit-Comment-Date: Fri, 22 Jun 2018 00:55:06 +0000
Gerrit-HasComments: Yes

Reply via email to