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

Jean-Daniel Cryans commented on HBASE-12774:
--------------------------------------------

Like we discussed, and because of HBASE-11008, it needs to be C which is your 
second option. Now, adding a C+W check isn't something we're doing anywhere 
else (I think?) and I'm not sure if it makes sense. The way things currently 
work, I'm not sure if it makes sense for a user to have C without having W.

> Fix the inconsistent permission checks for bulkloading.
> -------------------------------------------------------
>
>                 Key: HBASE-12774
>                 URL: https://issues.apache.org/jira/browse/HBASE-12774
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Srikanth Srungarapu
>            Assignee: Srikanth Srungarapu
>            Priority: Minor
>         Attachments: HBASE-12774.patch
>
>
> Three checks(prePrepareBulkLoad, preCleanupBulkLoad, preBulkLoadHFile)  are 
> being done while performing secure bulk load, and it looks the former two 
> checks for 'W', while the later checks for 'C'. After having offline chat 
> with [~jdcryans], looks like the inconsistency among checks were unintended. 
> So, we can address this multiple ways.
> * All checks should be for 'W'
> * All checks should be for 'C'
> * All checks should be for both 'W' and 'C'
> Posting the initial patch going by the first option. Open to discussion.



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

Reply via email to