hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jean-Daniel Cryans (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-12774) Fix the inconsistent permission checks for bulkloading.
Date Mon, 29 Dec 2014 23:55:13 GMT

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

Mime
View raw message