[
https://issues.apache.org/jira/browse/HBASE-6788?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13532674#comment-13532674
]
Ted Yu commented on HBASE-6788:
-------------------------------
Nice test.
{code}
+ public static interface BlockingAuthenticationService
...
+ public static class TokenServer extends TokenProvider
{code}
BlockingAuthenticationService and TokenServer are only used in
TestTokenAuthentication. I think they can be private.
{code}
+ // Creation of a HSA will force a resolve.
+ InetSocketAddress initialIsa = new InetSocketAddress(hostname, port);
{code}
'HSA' -> InetSocketAddress or simply ISA.
{code}
+ // ignore passed in controller -- it's always null
+ ServerRpcController serverController = new ServerRpcController();
{code}
Does the above mean similar logic needs to be used in non-test code ?
> Convert AuthenticationProtocol to protocol buffer service
> ---------------------------------------------------------
>
> Key: HBASE-6788
> URL: https://issues.apache.org/jira/browse/HBASE-6788
> Project: HBase
> Issue Type: Sub-task
> Components: Coprocessors
> Reporter: Gary Helmling
> Assignee: Gary Helmling
> Priority: Blocker
> Fix For: 0.96.0
>
> Attachments: HBASE-6788_2.patch, HBASE-6788_3.patch,
> HBASE-6788_4.patch, HBASE-6788.patch
>
>
> With coprocessor endpoints now exposed as protobuf defined services, we
> should convert over all of our built-in endpoints to PB services.
> AccessControllerProtocol was converted as part of HBASE-5448, but the
> authentication token provider still needs to be changed.
--
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