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

Francis Liu commented on HBASE-7066:
------------------------------------

More specifically if AccessController.preShutdown() throws an IOException, 
shutdown should fail since we cannot truly determine whether the shutdown 
request is authorized and thus should err on the side of caution and fail the 
request. 

There may be instances that preShutdown() hook exceptions can be ignored but 
that is not the case for AccessController which comes packaged with HBase thus 
it would make sense to guarantee that it works as advertised. We can address 
the previous scenario by fixing the coprocessor framework.
                
> Some HMaster coprocessor exceptions are being swallowed in try catch blocks
> ---------------------------------------------------------------------------
>
>                 Key: HBASE-7066
>                 URL: https://issues.apache.org/jira/browse/HBASE-7066
>             Project: HBase
>          Issue Type: Bug
>          Components: Coprocessors, security
>    Affects Versions: 0.94.2, 0.96.0
>            Reporter: Francis Liu
>            Assignee: Francis Liu
>            Priority: Critical
>         Attachments: HBASE-7066_94.patch, HBASE-7066_trunk.patch, 
> HBASE-7066_trunk.patch
>
>
> This is causing HMaster.shutdown() and HMaster.stopMaster() to succeed even 
> when an AccessDeniedException is thrown.

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