Opps. Forgot to send this file. > -----Original Message----- > From: Alan Cabrera [mailto:[EMAIL PROTECTED]] > Sent: Tuesday, January 15, 2002 10:48 PM > To: [EMAIL PROTECTED] > Subject: Re: [castor-dev] [JDO] Controlling instance creation in > ClassMold er > > > Thanks to everyone in this group. You guys were a tremendous help. > > > Regards, > Alan > > > -----Original Message----- > > From: Alan Cabrera [mailto:[EMAIL PROTECTED]] > > > > I'm kind of partial to InstanceFactory also. My updated > > classes and patch > > are on the way. > > > > > -----Original Message----- > > > From: David Blevins [mailto:[EMAIL PROTECTED]] > > > > > > How about a plain and simple InstanceFactory? > > > > > > > > > > -----Original Message----- > > > > From: Thomas Yip [mailto:[EMAIL PROTECTED]] > > > > > > > > Maybe DataObjectFactory? > > > > I am very bad at naming tough. > > > > > > > > > > > > > > > > > > > > Thank again. > > > > > > > > > > > > Thomas > > > > > > > > > > > > -----Original Message----- > > > > >From: Cabrera, Alan [mailto:[EMAIL PROTECTED]] > > > > >Sent: Tuesday, January 15, 2002 8:19 AM > > > > >To: [EMAIL PROTECTED] > > > > >Subject: Re: [castor-dev] [JDO] Controlling instance > creation in > > > > ClassMolder > > > > > > > > > >Gotcha. InstanceFactory sound good? > > > > > > > > > >I'll double check the OpenEJB side to make sure that it's > > > safe to remove > > > > >those methods. > > > > > > > > > >Alan > > > > > > > > > >-----Original Message----- > > > > >From: Thomas Yip [mailto:[EMAIL PROTECTED]] > > > > >Sent: Tuesday, January 15, 2002 1:47 AM > > > > >To: [EMAIL PROTECTED] > > > > >Subject: Re: [castor-dev] [JDO] Controlling instance > creation in > > > > >ClassMolder > > > > > > > > > > > > > > > > > > > >Hi Alan, > > > > > > > > > > > > > > >Thank you very much for your patch, especially appreciate that > > > > you send us > > > > >test cases as well. > > > > >I don't have time to run your test case yet. > > > > > > > > > >However, I have a minor enhancement request. First, I saw that > > > > you create a > > > > >new interface that extends CallbackInterceptor. However, I > > > think the new > > > > >interface would be better if it doesn't extends > > > CallbackInterceptor, > > > > because > > > > >it is actually not a extension of CallbackInterceptor, > > > > logically. Could you > > > > >please make it standalone? > > > > > > > > > >Also, the method > > > > > castor.jdo.Database.load( Class type, Object > > identity, Object > > > > object ), > > > > > castor.jdo.engine.DatabaseImpl( Class type, Object > > > identity, Object > > > > >object ), > > > > > > > > > >were added, half year ago, for openEJB integration > (although we > > > > later found > > > > >that openEJB can't take advantage of that). So, I am going to > > > > remove those > > > > >methods once your patch is committed. I am quite sure it > > > won't cause any > > > > >problem on your side, right? > > > > > > > > > > > > > > >Thank again Alan! Greatly appreciated. :-) > > > > > > > > > > > > > > > > > > > > > > > > >Thomas > > > > > > > > > > > > > > > > > > > >-----Original Message----- > > > > >>From: Alan Cabrera [mailto:[EMAIL PROTECTED]] > > > > >>Sent: Sunday, January 13, 2002 8:41 PM > > > > >>To: [EMAIL PROTECTED] > > > > >>Subject: Re: [castor-dev] [JDO] Controlling instance > creation in > > > > >ClassMolder > > > > >> > > > > >>Here are my files and test cases. > > > > >> > > > >
CallbacksInvoked.java
Description: Binary data
