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

stack commented on HBASE-18954:
-------------------------------

CoprocessorConfigurations.java is missing license and class comment on its 
intent (CoprocessorConfigurations is a generic name; you want to say something 
about who the configs are for, the audience? And yeah, as per Anoop, missing an 
Audience annotation).

Agree, *Host should be private (I see over in RSGroups where it is using Host 
to call CPs.... not sure how to undo that just yet -- smile).






> Make *CoprocessorHost classes private
> -------------------------------------
>
>                 Key: HBASE-18954
>                 URL: https://issues.apache.org/jira/browse/HBASE-18954
>             Project: HBase
>          Issue Type: Sub-task
>          Components: Coprocessors
>            Reporter: Appy
>            Assignee: Appy
>              Labels: incompatible
>             Fix For: 2.0.0-alpha-4
>
>         Attachments: HBASE-18954.master.001.patch
>
>
> Move out configuration name constants (into Coprocessor class?) and made Host 
> classes private.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to