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

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

[~apurtell]
When I saw the patch this doubt came to my mind too. I checked with other CP 
pre hooks like preMove(). Here also the CP can bypass the flow but reply wise 
there is no change. Ideally the reply should have a result flag and CP should 
be able to change this (as you said). CP can do some necessary action and 
bypass the core logic, still the operation may be treated as success only. What 
do you say Andrew? May be we can handle that in another JIRA as that will be of 
wider scope.
[~ram_krish] Rajesh's reasoning above make sense IMO. We can better document 
this API as part of HBASE-8076.

I am +1 for this patch.
                
> provide pre/post region offline hooks for HMaster.offlineRegion(). 
> -------------------------------------------------------------------
>
>                 Key: HBASE-7992
>                 URL: https://issues.apache.org/jira/browse/HBASE-7992
>             Project: HBase
>          Issue Type: Bug
>          Components: Coprocessors
>    Affects Versions: 0.95.0
>            Reporter: rajeshbabu
>            Assignee: rajeshbabu
>             Fix For: 0.98.0
>
>         Attachments: HBASE-7992_trunk.patch
>
>
> presently no hooks to provide access control to offline region in master.

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