Michael Ho has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12106 )
Change subject: Add "network_plane" as part of ConnectionId ...................................................................... Add "network_plane" as part of ConnectionId The motivation for doing so is to allow N services on the same host to be multiplexed on M different connections. For instance, a server may host multiple KRPC based services: one for control command and one for data transfer. Separating the connections between the control channel and the data channel prevents unnecessary delays of the control commands due to being stuck behind large data transfers from client to server. By default, the network_plane of a new ConnectionId is not set. A user can change it to a different value by calling Proxy::set_network_plane() on the ConnectionId. Change-Id: I6767e631fd9530ea54f5ed63ff4c8c179ab216b2 Reviewed-on: http://gerrit.cloudera.org:8080/11681 Reviewed-by: Adar Dembo <a...@cloudera.com> Tested-by: Kudu Jenkins Reviewed-on: http://gerrit.cloudera.org:8080/12106 Reviewed-by: Thomas Marshall <thomasmarsh...@cmu.edu> Tested-by: Michael Ho <k...@cloudera.com> --- M be/src/kudu/rpc/connection_id.cc M be/src/kudu/rpc/connection_id.h M be/src/kudu/rpc/messenger.h M be/src/kudu/rpc/proxy.cc M be/src/kudu/rpc/proxy.h M be/src/kudu/rpc/reactor.h M be/src/kudu/rpc/rpc-test.cc 7 files changed, 138 insertions(+), 31 deletions(-) Approvals: Thomas Marshall: Looks good to me, approved Michael Ho: Verified -- To view, visit http://gerrit.cloudera.org:8080/12106 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I6767e631fd9530ea54f5ed63ff4c8c179ab216b2 Gerrit-Change-Number: 12106 Gerrit-PatchSet: 2 Gerrit-Owner: Michael Ho <k...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Michael Ho <k...@cloudera.com> Gerrit-Reviewer: Thomas Marshall <thomasmarsh...@cmu.edu>