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

Reply via email to