Doh! I just updated the whole source tree and I see what you
did. I can set the InstanceFactory at the JDO level. I see that you
really did a lot of work to get this in for me.
Thanks!
If
you're ever Manhattan way, I will buy you a beer or two.
Regards,
Alan
-----Original Message-----
From: Alan Cabrera [mailto:[EMAIL PROTECTED]]
Sent: Thursday, January 24, 2002 11:40 PM
To: [EMAIL PROTECTED]
Subject: [castor-dev] [JDO] InstanceFactory checkinHey 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
