Thanks for the confirmations, Jean-Baptiste! :-) On Fri, Mar 5, 2010 at 8:17 AM, Jean-Baptiste BRIAUD -- Novlog < j-b.bri...@novlog.com> wrote:
> I can confirm the bug #1020 is corrected in the 2.0.0 beta2. > I can also confirm the migration path from the 1.2.2 is really painless but > can't be optimized : nothing to do :-) > I only had to copy the new jar files. > > I even think to keep that version but I'll need more test as we are at the > end of an iteration cycle. > > OpenJPa power ! > > On Mar 2, 2010, at 15:30 , Kevin Sutter wrote: > > > Sounds good, Jean-Baptiste. We have resolved the problem as identified > by > > the JIRA and the associated testcase. So, hopefully when you have the > > chance to move up to OpenJPA 2.0.x, this particular problem will be > resolved > > for you. > > > > Just as an FYI... We have been very conscientious about testing JPA 1.0 > > applications with our JPA 2.0 runtime. There have been a couple of minor > > items as we attempt to merge OpenJPA 1.x features into the JPA 2.0 > > implementation (ie. like our original pessimistic lock manager and newly > > defined pessimistic lock manager). But, even in those cases, we have > > attempted to document the differences and provide compatibility switches > to > > revert to old behavior. Hopefully, your migration to OpenJPA 2.0.x will > be > > smooth. > > > > Good luck and thanks for the feedback, > > Kevin > > > > On Tue, Mar 2, 2010 at 6:44 AM, Jean-Baptiste BRIAUD -- Novlog < > > j-b.bri...@novlog.com> wrote: > > > >> 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 <kwsut...@gmail.com> > >> 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 < > >>>> j-b.bri...@novlog.com> 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 < > >>>>>> j-b.bri...@novlog.com> 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/> > >> <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/> > >> <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/ > > > >> < > >> > 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/ > > > >> < > >> > 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/ > > > >> < > >> > 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 > >>>>>>>> > >>>>>>> > >>>>>>> > >>>>> > >>>>> > >>>> > >> > >> > >