sentry-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Prasad Mujumdar (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (SENTRY-332) A role may got empty privilege, although the role have some privileges
Date Tue, 15 Jul 2014 02:39:07 GMT

    [ https://issues.apache.org/jira/browse/SENTRY-332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14061579#comment-14061579
] 

Prasad Mujumdar commented on SENTRY-332:
----------------------------------------

[~dapengsun] I guess that make sense. DataNucleus seems to have some complex logic to fetch
the complex object conditionally. The patch itself look fine to me. We should keep an eye
on the performance hit as every  role fetch will now cause additional joins with the mapping
table.

> A role may got empty privilege, although the role have some privileges
> ----------------------------------------------------------------------
>
>                 Key: SENTRY-332
>                 URL: https://issues.apache.org/jira/browse/SENTRY-332
>             Project: Sentry
>          Issue Type: Bug
>    Affects Versions: 1.4.0
>            Reporter: Dapeng Sun
>            Assignee: Dapeng Sun
>             Fix For: 1.4.0
>
>         Attachments: SENTRY-332.patch
>
>
> MSentryPrivilege is an embedded field of MSentryRole, When query a MSentryRole that contains
MSentryPrivilege as embedded fields, the embedded field MSentryPrivilege may returned as null.
> For example:
> {code:borderStyle=solid}
> MSentryRole mRole = getMSentryRole(pm, roleName);
> if (mRole != null) {
>      mRole.getPrivileges(); // the privileges may be null, although the role have privileges
> }
> {code}
> Solution:
> JDO defaults to lazy loading ´╝îMSentryPrivilege should be  eager-loading .



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Mime
View raw message