Hello Alexey Serbin, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/17969
to look at the new patch set (#2).
Change subject: [rpc] re-add methods_by_name()
......................................................................
[rpc] re-add methods_by_name()
This patch reverts the removal of the methods_by_name() accessor method
from patch 7f794e3ac8c389af4c74faeac11274e4ef0a55c0. Turns out it may be
used outside the project, in the Apache Impala project[1]. To maintain
consistency between the two projects, this re-adds the method.
This patch also adds a simple test with this context to deter further
removals of the call.
[1]
https://github.com/apache/impala/blob/b28da054f3595bb92873433211438306fc22fbc7/be/src/rpc/impala-service-pool.cc#L77
Change-Id: I2cbccdbbd119ebd53544d951e45b8bb4d34abae7
---
M src/kudu/rpc/rpc-test.cc
M src/kudu/rpc/service_if.h
2 files changed, 14 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/69/17969/2
--
To view, visit http://gerrit.cloudera.org:8080/17969
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I2cbccdbbd119ebd53544d951e45b8bb4d34abae7
Gerrit-Change-Number: 17969
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
