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

Appy commented on HBASE-19235:
------------------------------

CPHosts use CPEnv to load and manage CP instances.
The functions startup(), shutdown(), loadSequence(), getPriority(), 
getInstance, etc are all related to management of a loaded Coprocessor. No 
observer needs them. Also we'll want to freely add/remove functions here. 

bq. Its sub interfaces are exposed with 
LimitedPrivate(HBaseInterfaceAudience.COPROC). So ideally all the functions in 
this are.
Is this how our audience annotations are interpreted? To me, i always saw it 
like, IA.Foo on a class/interface is for that class/interface only, it doesn't 
translate to parent.
In fact in this case, we have explicitly marked CPEnv as Private, so there 
shouldn't be any confusion?

> CoprocessorEnvironment should be exposed to CPs
> -----------------------------------------------
>
>                 Key: HBASE-19235
>                 URL: https://issues.apache.org/jira/browse/HBASE-19235
>             Project: HBase
>          Issue Type: Sub-task
>          Components: Coprocessors
>    Affects Versions: 2.0.0-alpha-4
>            Reporter: Anoop Sam John
>            Assignee: Anoop Sam John
>            Priority: Minor
>             Fix For: 2.0.0-beta-1
>
>         Attachments: HBASE-19235.patch
>
>
> Its sub interfaces are exposed with 
> LimitedPrivate(HBaseInterfaceAudience.COPROC).  So ideally all the functions 
> in this are.  Better we mark CoprocessorEnvironment also as CP exposed to 
> avoid confusion.



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

Reply via email to