Duo Zhang created HBASE-30534:
---------------------------------

             Summary: The filterUser logic should not be placed in 
PermissionStorage
                 Key: HBASE-30534
                 URL: https://issues.apache.org/jira/browse/HBASE-30534
             Project: HBase
          Issue Type: Improvement
            Reporter: Duo Zhang
            Assignee: Duo Zhang


HBase's ACL model supports group, which is stored as
{noformat}
@GROUP_NAME
{noformat}

in acl table.

And when checking whether a user has proper permissions or getting the 
permissions for a user, we need to expand the group to a list of users. The 
filterUser parameter is used in this case.

But PermissionStorage is designed as class to communicate with the acl table, 
so putting the filterUser logic in it, will mess up the logic for 
implementation grant/revoke.

So I prefer we do a refactoring, to leave PermissionStorage as simple as 
possible, and move the filterUser logic to upper layer, which could make the 
code easier to understand.



--
This message was sent by Atlassian Jira
(v10.3.26#10030026)

Reply via email to