openjpa-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Kevin Sutter (JIRA)" <>
Subject [jira] Commented: (OPENJPA-370) LoadFetchGroup annotation was not recognized during the fetch1
Date Fri, 28 Sep 2007 13:58:50 GMT


Kevin Sutter commented on OPENJPA-370:

I'm looking at your patch and the comments in this Issue.  The Patch 2 approach looks good
to me.  I suggest we go ahead with this patch for both 1.0.x and 1.1.0.  If there are tweaks
(desired or required) to the implementation, then we'll deal with them.  At least with this
patch, we now have a working LoadFetchGroup annotation.

Question concerning the tests.  They all seem to run just fine, but is the current package
the proper location (org.apache.openjpa.persistence.query)?  I'm thinking a separate fetchgroup
package would be better.  And, we 're bound to add more tests along the way.  If that's okay
with you, I'll do the move, you don't have to supply another patch.  I will also add the required
license header on the testcases.


> LoadFetchGroup annotation was not recognized during the fetch1
> --------------------------------------------------------------
>                 Key: OPENJPA-370
>                 URL:
>             Project: OpenJPA
>          Issue Type: Bug
>          Components: kernel
>    Affects Versions: 1.0.1, 1.1.0
>            Reporter: Teresa Kan
>            Assignee: Teresa Kan
>         Attachments: OPENJPA_370_2.patch,
> Employee class has a LoadFetchGroup annotation defined on the Rating field, when getRating
was called, the address should be returned also. However, openjpa did not handle the LoadFetchGroup
correctly, therefore, address was not eargly fetched.
> public class FGEmployee{
>     @Id
>     private int id;
>     @OneToOne(fetch=FetchType.LAZY) 
>     private FGAddress address;
>     @Basic(fetch=FetchType.LAZY)
>     @LoadFetchGroup("AddressFetchGroup")
>     private String rating;
>     @ManyToOne(fetch=FetchType.LAZY)
>     private FGManager manager;
> ..
> }

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message