[jira] [Commented] (HBASE-12774) Fix the inconsistent permission checks for bulkloading.

["Jean-Daniel Cryans (JIRA)" <ji...@apache.org>]

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