Hi Kevin, You were clear, I didn't understood your previous response as a promise :-) That's a very good news for us that this bug is corrected. This led us to add extra code everywhere a collection is used. So, really, I love the idea to remove that code using the version 2.
I didn't try 2.0 so I'm not trying to say the bug is still there. I'll give a try to version 2 but its quite risky for us now as we are ending an iteration. I hope I'll be able to reach at least the case of collection to confirm the bug correction since I won't be able to change our code in this ending iteration cycle. On Mar 2, 2010, at 00:24 , Kevin Sutter wrote: > Hi Jean-Baptiste, > It was just pointed out to me that OPENJPA-1020 was resolved in trunk back > on Jan 04, 2010 by Fay Wang. I missed that commit message on the JIRA. So, > this problem has been resolved in trunk and should be part of both the > 2.0.0-beta and 2.0.0-beta2. > > Are you indicating that this problem still exists for you on trunk (beta or > beta-2)? If so, then we should document that in the JIRA. > > Sorry for the confusion, > Kevin > > On Mon, Mar 1, 2010 at 11:21 AM, Kevin Sutter <[email protected]> wrote: > >> Hi Jean-Baptiste, >> Sorry if I mis-represented the approach, but I never promised to resolve >> this JIRA Issue before committing the Beta2 release. I said that we would >> take a look at the referenced JIRA and try to work with the patch originator >> to see if we could resolve this before we do a final 2.0.0 release. The >> BetaX releases should not be required to have certain JIRAs fixed. Unless >> we're talking about regressions. But, in this case, the problem has been >> around for a while and, thus, should not hold up a BetaX release. >> >> Kevin >> >> >> On Mon, Mar 1, 2010 at 10:17 AM, Jean-Baptiste BRIAUD -- Novlog < >> [email protected]> wrote: >> >>> Hi, >>> >>> I've seen tge beta2 is out but the bug is still not corrected according to >>> issue tracker. >>> http://issues.apache.org/jira/browse/OPENJPA-1020 >>> >>> Apparently, a patch is already there. I'm not sure what to do and if >>> someone did something. >>> >>> Any news ? >>> >>> This bug make PersistentCollection not working with Eager settings. >>> The only workaround is to rely on that evil lazy loading so you have to >>> call a getter for an extra unuseful call to the DB just for that. >>> >>> Who is Ted Leung ? What could we do to make that bug corrected ? >>> >>> Thanks for your help. >>> >>> On Feb 25, 2010, at 15:46 , Kevin Sutter wrote: >>> >>>> You are correct. It doesn't look like >>>> OPENJPA-1020<http://issues.apache.org/jira/browse/OPENJPA-1020>is >>>> resolved for this Beta2 driver. Hopefully, we can get this resolved >>>> before the next Betan/GA driver for 2.0.0. I noticed that both a >>> testcase >>>> and a potential patch have been provided by Ted Leung. Maybe we can >>> hook up >>>> Ted with a committer to get this resolved. >>>> >>>> Kevin >>>> >>>> On Thu, Feb 25, 2010 at 3:59 AM, Jean-Baptiste BRIAUD -- Novlog < >>>> [email protected]> wrote: >>>> >>>>> This bug doesn't look like to be corrected : >>>>> http://issues.apache.org/jira/browse/OPENJPA-1020 >>>>> >>>>> Could you please confirm the status of than OPENJPA-1020 ? >>>>> >>>>> I would be tempted to vote -1 as that bug is very annoying >>>>> >>>>> On Feb 24, 2010, at 23:55 , Donald Woods wrote: >>>>> >>>>>> I've staged a release candidate for OpenJPA 2.0.0 Beta 2 based on >>>>>> r915978 of the code in the following svn tag: >>>>>> https://svn.apache.org/repos/asf/openjpa/tags/2.0.0-beta2/ >>>>>> >>>>>> Release Notes: >>>>>> >>>>> >>> http://svn.apache.org/viewvc/openjpa/tags/2.0.0-beta2/openjpa-project/RELEASE-NOTES.html?view=co >>>>>> >>>>>> Maven staging repo: >>>>>> http://people.apache.org/~dwoods/openjpa/2.0.0-beta2/staging-repo/<http://people.apache.org/%7Edwoods/openjpa/2.0.0-beta2/staging-repo/> >>> <http://people.apache.org/%7Edwoods/openjpa/2.0.0-beta2/staging-repo/> >>>>>> >>>>>> Site staging: >>>>>> http://people.apache.org/~dwoods/openjpa/2.0.0-beta2/staging-site/<http://people.apache.org/%7Edwoods/openjpa/2.0.0-beta2/staging-site/> >>> <http://people.apache.org/%7Edwoods/openjpa/2.0.0-beta2/staging-site/> >>>>>> >>>>>> Distribution artifacts: >>>>>> >>>>> >>> http://people.apache.org/~dwoods/openjpa/2.0.0-beta2/staging-site/apache-openjpa/downloads/<http://people.apache.org/%7Edwoods/openjpa/2.0.0-beta2/staging-site/apache-openjpa/downloads/> >>> < >>> http://people.apache.org/%7Edwoods/openjpa/2.0.0-beta2/staging-site/apache-openjpa/downloads/ >>>> >>>>>> >>>>>> User's Guide: >>>>>> >>>>> >>> http://people.apache.org/~dwoods/openjpa/2.0.0-beta2/staging-site/apache-openjpa/docs/<http://people.apache.org/%7Edwoods/openjpa/2.0.0-beta2/staging-site/apache-openjpa/docs/> >>> < >>> http://people.apache.org/%7Edwoods/openjpa/2.0.0-beta2/staging-site/apache-openjpa/docs/ >>>> >>>>>> >>>>>> API Docs: >>>>>> >>>>> >>> http://people.apache.org/~dwoods/openjpa/2.0.0-beta2/staging-site/apidocs/<http://people.apache.org/%7Edwoods/openjpa/2.0.0-beta2/staging-site/apidocs/> >>> < >>> http://people.apache.org/%7Edwoods/openjpa/2.0.0-beta2/staging-site/apidocs/ >>>> >>>>>> >>>>>> The eventual landing page for Beta 2 on the wiki: >>>>>> >>> http://cwiki.apache.org/confluence/display/openjpa/OpenJPA+2.0.0+Beta+2 >>>>>> >>>>>> >>>>>> The RAT and IANAL build checks passed. >>>>>> The openjpa and openjpa-all shaded jar content looks good. >>>>>> The test-base, test-sources and tests jars are in the staging repo. >>>>>> The build passed the JPA 2.0 TCK. >>>>>> >>>>>> >>>>>> The vote will be open for 72 hours or less. >>>>>> >>>>>> [ ] +1 >>>>>> [ ] 0 >>>>>> [ ] -1 (and reason why) >>>>>> >>>>>> >>>>>> Thanks, >>>>>> Donald >>>>>> >>>>> >>>>> >>> >>> >>
