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

Ted Yu commented on HBASE-7042:
-------------------------------

Line below is not needed:
{code}
+ * Copyright 2010 The Apache Software Foundation
{code}
{code}
+  private Map<String, Class<? extends CoprocessorProtocol>>
+      protocolHandlerNames = Maps.newHashMap();
{code}
The above is a map. How about naming it protocolsByName ?
MasterExec is referenced in two places, such as:
{code}
+   * be available for handling
+   * {@link 
HMaster#execCoprocessor(org.apache.hadoop.hbase.client.coprocessor.MasterExec)} 
calls.
{code}
Please replace it with Exec.

                
> Master Coprocessor Endpoint
> ---------------------------
>
>                 Key: HBASE-7042
>                 URL: https://issues.apache.org/jira/browse/HBASE-7042
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>             Fix For: 0.96.0
>
>         Attachments: HBASE-7042_94_2.patch, HBASE-7042_94.patch
>
>
> Having support for a master coprocessor endpoint would enable developers to 
> easily extended HMaster functionality/features. As is the case for region 
> server grouping.

--
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