Re: [castor-dev] [JDO] InstanceFactory checkin

[Thomas Yip]

Hum… maybe I missed an openEJB concerns then.   Can you tell me why the following doesn’t work for you?   OpenEJBInstanceFactoryAndCallBack icb    = new OpenEJBInstanceFactoryAndCallBack icb( … ); JDO jdo = new JDO(); jdo.setInstanceFactory( icb ); jdo.setCallbackInterceptor( icb );   I think it works exactly the same except it allows both call back and instance to be in the same class, or to be two.   Let me know, I am sure we can find some solution.       Thomas     -----Original Message----- >From: Alan Cabrera [mailto:[EMAIL PROTECTED]] >Sent: Thursday, January 24, 2002 8:40 PM >To: [EMAIL PROTECTED] >Subject: [castor-dev] [JDO] InstanceFactory checkin > >Hey Tom, I saw that my code for an InstanceFactory was checked in. Thanks!  >  >There's one teeny problem.  You added another set method, setInstanceFactory(), instead of using setCallback() to the TransactionContext class.  I think it would be a problem for OpenEJB and maybe others who wish to provide an InstanceFactory.  Doing it my previous way, you only had to set the call back at the JDO level once, which is awfully convenient.  Doing it the new way, I have to dig up the transaction context and set it with my InstanceFactory every time.  >  >At first glance, I think that it will take a ton of code changes in OpenEJB to use the new way, whereas it was a few line changes before.  I think that others will have the same difficulty in using the new paradigm also. >  >  >Regards, >Alan >