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

Anoop Sam John commented on HBASE-19235:
----------------------------------------

Appy.  IMHO #2 is not at all good.  Am not sure any other place we have like 
super interface is private and sub type as LP or public.  I did not check.  If 
so, we have to fix. I raised this issue after seeing a patch in Phoenix.   The 
sub interfaces are LP exposed. So any functions available to it, can be called 
by user.  Else the CP user has to really check in which interface it is 
declared and see its declaration. It will be very difficult task and confusing 
for sure.
Ya we were having CPEnv interface as private till now. But we can cleanup now. 
Better late than never.
I understood Chia comment to avoid the instance of check. Let me see. I thought 
we can make it simple. Like in old way.  Let me see.

> 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, HBASE-19235_V2.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