[ 
https://issues.apache.org/jira/browse/HBASE-6789?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13497624#comment-13497624
 ] 

Gary Helmling commented on HBASE-6789:
--------------------------------------

[[email protected]] You could check out the HBASE-6786 changes or take a 
look at the {{TestCoprocessorEndpoint.testCoprocessorService()}} test I added 
as part of HBASE-5448.  I also converted a lot of {{TestAccessController}} to 
the PB service usage, so that may be helpful as well.

I'm also prepping a patch for HBASE-6788 (AuthenticationProtocol conversion) 
that I should have up soon.  The Token serialization there is a bit funky 
(since that's upstream Hadoop code), but the rest is a pretty straight 
CoprocessorProtocol -> CoprocessorService change.
                
> Convert test CoprocessorProtocol implementations to protocol buffer services
> ----------------------------------------------------------------------------
>
>                 Key: HBASE-6789
>                 URL: https://issues.apache.org/jira/browse/HBASE-6789
>             Project: HBase
>          Issue Type: Sub-task
>          Components: Coprocessors
>            Reporter: Gary Helmling
>             Fix For: 0.96.0
>
>
> With coprocessor endpoints now exposed as protobuf defined services, we 
> should convert over all of our built-in endpoints to PB services.
> Several CoprocessorProtocol implementations are defined for tests:
> * ColumnAggregationProtocol
> * GenericProtocol
> * TestServerCustomProtocol.PingProtocol
> These should either be converted to PB services or removed if they duplicate 
> other tests/are no longer necessary.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to