[ https://issues.apache.org/jira/browse/HBASE-7066?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13489820#comment-13489820 ]
Andrew Purtell commented on HBASE-7066: --------------------------------------- bq. Have we ever considered making security first class in 0.96 (not a coprocessor, though still switchable)? I'm not in favor of this approach unless we want HBASE-6222, specifically the KV labeling part, in which case the changes are so invasive anyway we might as well move everything into core and furthermore reimplement access control on top of labeling. bq. Change stopMaster() and shutdown() signature with "throws AccessDeniedException" I think it will be a little weird to have these two methods throw a more specific signature than IOE where everywhere else we have IOE. The larger issue of (ab)use of IOE is a major refactoring. Also, I think AccessDeniedException should remain in the security package until security is otherwise not encapsulated there. > 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