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
