[
https://issues.apache.org/jira/browse/HBASE-11251?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14008594#comment-14008594
]
Anoop Sam John commented on HBASE-11251:
----------------------------------------
+ HTable table = new HTable(conf, tableName);
The table is just used to get the tabelName which we already have. We can
avoid this HTable creation and so the close in finally. This is there in 2
places in the patch.
{code}
- AccessControlClient.grant(conf, tableName, userOwner.getShortName(),
COLUMN_FAMILY,
+ try {
+ AccessControlClient.grant(conf, table.getName(),
userOwner.getShortName(), null,
{code}
Any reason why the specific CF grant is changed in this patch?
> LoadTestTool should grant READ permission for the users that are given READ
> access for specific cells
> -----------------------------------------------------------------------------------------------------
>
> Key: HBASE-11251
> URL: https://issues.apache.org/jira/browse/HBASE-11251
> Project: HBase
> Issue Type: Bug
> Affects Versions: 0.98.2
> Reporter: ramkrishna.s.vasudevan
> Assignee: ramkrishna.s.vasudevan
> Fix For: 0.99.0, 0.98.3
>
> Attachments: HBASE-11251_0.98.patch, HBASE-11251_trunk.patch
>
>
> In 0.98.2 onwards the AccessControlFilter
> {code}
> case CHECK_CELL_FIRST: {
> LOG.info("Am coming here for cell first strategy");
> if (authManager.authorize(user, table, cell, Permission.Action.READ)
> &&
> authManager.authorize(user, table, family, qualifier,
> Permission.Action.READ)) {
> LOG.info("Returning include");
> return ReturnCode.INCLUDE;
> }
> {code}
> expects a READ permission on the table for those Users that are granted READ
> permission on the cell level.
> In 0.98.1
> {code}
> if (authManager.authorize(user, table, cell, cellFirstStrategy,
> Permission.Action.READ)) {
> return ReturnCode.INCLUDE;
> }
> {code}
> So from 0.98.2 onwards IntegrationTestIngestWithACL was failing. Hence this
> JIRA is targeted to correct the behaviour and make the IT work again.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
