Yes, thanks for asking, Sean. One way you can test this is to apply your
patch locally, have Phoenix point to your new local version, and the just
run our unit test suite with "mvn verify".
James
On Mon, Mar 6, 2017 at 3:26 PM, Josh Elser <[email protected]> wrote:
> Hey Sean,
>
> Thanks for taking a moment to ask!
>
> I did a quick grep over some branches for `RpcServerInterface` and I don't
> see any usages. The only thing I could find (and, that I was aware of) was
> some use of the RpcScheduler classes. It doesn't seem like this change
> would cause any heartburn.
>
>
> Sean Busbey wrote:
>
>> Hi Folks!
>>
>> If someone could take a look at HBASE-17725, I'd appreciate it.
>>
>> short version: The RpcServiceInterface API has added a method. That
>> interface is marked as IA.LimitedPrivate for Coprocessors generally and
>> Phoenix specifically. The change is already in two HBase 1.1 releases, but
>> 1.2.5 would be the first 1.2 release to include it.
>>
>> I'd like to know if the way that Phoenix makes use of this interface will
>> be broken by the change, or if y'all are already compensating for it.
>>
>