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

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

bq.public boolean preBackupTables
What is the use of this return type. Seems that will be unused. I can not see 
in MasterCoprocessorHost the return of the pre hook is used.  The bypass usage 
is enough?  Or we should return a Pair as the real op doing?  This is inline 
with preScannerOpen() and all..  To pre hook we pass the type and table names. 
Other details also needed?  Like request.getTargetRootDir()?
For post hook, we need to pass the Pair result of call master.backupTables ()?  
So we need Global Admin privilege to do the backup op.   Table admin can do 
back up of that table?



> Add ACL for requesting table backup
> -----------------------------------
>
>                 Key: HBASE-15590
>                 URL: https://issues.apache.org/jira/browse/HBASE-15590
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Ted Yu
>            Assignee: Ted Yu
>         Attachments: 15590.v1.patch
>
>
> This issue adds necessary coprocessor hooks for table backup request along 
> with enforcing permission check in AccessController through the new hooks.
> To perform backup, admin privilege is required in secure deployment. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to