hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jonathan Hsieh (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-11318) Classes in security subpackages missing @InterfaceAudience annotations.
Date Tue, 10 Jun 2014 21:33:01 GMT

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

Jonathan Hsieh commented on HBASE-11318:

I did not, thanks for the links. I only look at the classes that are in *.security.* packages.
 I think we need [~apurtell] to take a quick look on the TablePermission and UserPermission
and resolve the api in consistency.  I"ll move this to a subissue of HBASE-10671 for now and
we can close them all out when they get it.

> Classes in security subpackages missing @InterfaceAudience annotations.
> -----------------------------------------------------------------------
>                 Key: HBASE-11318
>                 URL: https://issues.apache.org/jira/browse/HBASE-11318
>             Project: HBase
>          Issue Type: Bug
>          Components: security
>    Affects Versions: 0.99.0,, 0.98.3
>            Reporter: Jonathan Hsieh
>            Assignee: Jonathan Hsieh
>             Fix For: 0.99.0, 0.98.4
>         Attachments: hbase-11318.patch
> I was reading some of the security related code and noticed that many of the security
related classes lack @InterfaceAudience markings.
> WIth the current api I believe all but Permission should be Private.  With the introduction
of cell level ACL's Permission must be public because it is now exposed in the Mutation setACL
> There is an inconsistency with the Mutation ACL -- the acl setters take Permission instances
but the getter returns byte[]'s.  As a follow on issue we could change the signature of Mutation.setACL
so we don't have to expose the Permission class and convert it to be byte[], or change the
getter to return an exposed Permission instance.
> [1] http://hbase.apache.org/devapidocs/org/apache/hadoop/hbase/client/Mutation.html#setACL(java.util.Map)

This message was sent by Atlassian JIRA

View raw message