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

Andrew Purtell commented on HBASE-18127:
----------------------------------------

Aside from functional improvements, the big concern is interface compatibility 
impact. I like how the only interface that CP implementers might use that you 
have changed is the LP(Coproc)/Evolving interface RowProcessor. 

OperationCoprocessorContext could just be called OperationContext? Do we need 
SimpleOperationCoprocessorContext? Will there ever be another kind? Consider 
making OperationContext a class, like ObserverContext is a class. 



> Allow regionobserver to optionally skip postPut/postDelete when 
> postBatchMutate was called
> ------------------------------------------------------------------------------------------
>
>                 Key: HBASE-18127
>                 URL: https://issues.apache.org/jira/browse/HBASE-18127
>             Project: HBase
>          Issue Type: New Feature
>            Reporter: Lars Hofhansl
>            Assignee: Abhishek Singh Chouhan
>         Attachments: HBASE-18127.master.001.patch
>
>
> Right now a RegionObserver can only statically implement one or the other. In 
> scenarios where we need to work sometimes on the single postPut and 
> postDelete hooks and sometimes on the batchMutate hooks, there is currently 
> no place to convey this information to the single hooks. I.e. the work has 
> been done in the batch, skip the single hooks.
> There are various solutions:
> 1. Allow some state to be passed _per operation_.
> 2. Remove the single hooks and always only call batch hooks (with a default 
> wrapper for the single hooks).
> 3. more?
> [~apurtell], what we had discussed a few days back.



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

Reply via email to