Alexey Serbin has submitted this change and it was merged. Change subject: [rpc] reopen_outbound_connections mode for reactor ......................................................................
[rpc] reopen_outbound_connections mode for reactor Introduced a test mode for the RPC ReactorThread: reopen already existing idle outbound connection upon making another remote call to the same server. This is a groundwork for follow-up patches which contain tests requiring connection negotiation to be done upon every RPC call. Added a couple of new ReactorThread metrics to count total number of server- and client-side connections open during ReactorThread lifetime. Added unit test to cover the newly introduced functionality. Also, did a minor clean-up on the ReactorThread code and its inline documentation. Change-Id: I71ada660d8c92de1813a261e221f73017c2c764a Reviewed-on: http://gerrit.cloudera.org:8080/6710 Tested-by: Kudu Jenkins Reviewed-by: Dan Burkert <danburk...@apache.org> --- M src/kudu/rpc/connection.h M src/kudu/rpc/messenger.h M src/kudu/rpc/reactor.cc M src/kudu/rpc/reactor.h M src/kudu/rpc/rpc-test.cc 5 files changed, 102 insertions(+), 24 deletions(-) Approvals: Dan Burkert: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/6710 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I71ada660d8c92de1813a261e221f73017c2c764a Gerrit-PatchSet: 5 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
