Thanks, Oleg. The commit you've done works great ! Regards, Guillaume
> -----Message d'origine----- > De : Oleg Nitz [mailto:[EMAIL PROTECTED] > Envoy� : lundi 16 f�vrier 2004 01:38 > � : OJB Users List > Objet : Re: Bug in batch mode > > > Hi Armin and Guillaume, > > Thank you very much for the test, which made easy to understand > what happends. > That was not really the bug in batch mode, but rather the feature > in PB API, > which I never liked: the same method store() is used for two operations, > INSERT and UPDATE, and in the case of batch mode additional > SELECT may not > give a correct choice, namely if the object has been deleted, but > the batch > hasn't been executed yet. I've made the algorithm of choice of INSERT or > DELETE a bit more clever and that solved the problem. > I've committed the fix to CVS HEAD. Should I apply the fix to the > branch-1.0? > > Regards, > Oleg > > On Friday 13 February 2004 19:00, Armin Waibel wrote: > > Hi Guillaume, > > > > ok, I get all your files to run your test. > > > > Have to make some modifications: > > > > - In metadata declaration of SubObject field 'mainid' you set access to > > anonymous, this is not recommended here. Anonymous fields can only safe > > used in conjunction with 1:1 references (The docs are not clear here, > > but AFAIK Brian is working on an update). > > > > - In metadata declaration of SubObject field 'id' you declare an > > attribute anonymous="true", such an attribute does not exist (AFAIK). > > > > > > *testDeleteInsert()* > > Seems to be a bug in batch handling. Without batch mode enabled test > > pass, with enabled batch mode test fails. > > > > > > > > *testEquals()* > > When I delete the object before storing the new instance with same PK > > test pass > > > > broker.beginTransaction(); > > broker.store(main1); > > // delete object before add new instance with same PK > > broker.delete(main1); > > broker.store(main2); > > broker.commitTransaction(); > > > > > > > > *testDelete()* > > test pass. > > > > You can find the TestCase in test suite (CVS HEAD) called > > ...broker.BatchModeTest. Please review my changes. > > > > regards, > > Armin > > > > Guillaume Nodet wrote: > > > I'm splitting it because i have problems when attaching a zip file. > > > > > > > > > > ------------------------------------------------------------------------ > > > > > > --------------------------------------------------------------------- > > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, e-mail: [EMAIL PROTECTED] > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
