> -----Original Message-----
> From: [EMAIL PROTECTED]
> [mailto:[EMAIL PROTECTED]]On Behalf Of marc
> fleury
> Sent: Monday, June 11, 2001 4:08 PM
> To: [EMAIL PROTECTED]
> Subject: RE: [JBoss-dev] CVS update:
> jboss/src/main/org/jboss/ejb/plugins
> EntitySynchronizationInterceptor.java EntityInstanceInterceptor.java
> EntityInstanceCache.java AbstractInstanceCache.java
>
>
> bill,
>
> I would really check all of the access to the state of the ctx. Since the
> ctx is multithreaded we need to make sure that all the operations that
> access it are kosher.
>
> I remember going through the excercise for the earlier versions and asking
> simone to do the same.
>
> Simply do a search on the methods that access the Context and
> make sure you
> synchronize correctly.
>
> I remember that the cache passivation REALLY made things more complex
>
Off the top of my head, the first one I can think of it ctx.setTransaction
that is done in EntityContainer.ContainerInterceptor.invoke. In my check-in
I do this within EntityInstanceInterceptor.java instead. I also undo the
setTransaction if an InstanceSynchronization is not registered with the TM.
Should setTransaction be removed from
EntityContainer.ContainerInterceptor.invoke, now that I do the association
in EntityInstanceInterceptor?
I'll look throw ctx access more thoroughly later tonight. I have "paid"
work I must accomplish before hand....
Bill
BTW, sorry again about the commit to mainline instead of the branch....
[snip]
_______________________________________________
Jboss-development mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/jboss-development
