[infinispan-dev] exceptions and locking policy

[Mircea Markus]

On 6 Sep 2010, at 14:33, Manik Surtani wrote:

> 
> On 6 Sep 2010, at 14:28, Mircea Markus wrote:
> 
>>> I notice you've ripped this logic out of the InvocationContextInterceptor in branches/4.2.x
>>> 
>>> http://fisheye.jboss.org/browse/Infinispan/branches/4.2.x/core/src/main/java/org/infinispan/interceptors/InvocationContextInterceptor.java?r1=2273&r2=2323&u=3#l45
>>> 
>>> Note that this is incorrect since, as I pointed out, you will miss exceptions thrown between the InvocationContextInterceptor and the LockingInterceptor.
>> This code was committed before this discussion to take place. I needed it in because couldn't work on DLD without it). 
>> It wil be addressed in  ISPN-629
> 
> Ok, just be careful when rolling it back/reimplementing - it's tricky stuff.  A few extra tests around it would be a good idea as well.  :)
Of course- that's why I added these [1] before making the changes :) 
[1] http://fisheye.jboss.org/browse/Infinispan/branches/4.2.x/core/src/test/java/org/infinispan/tx/timeout

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/infinispan-dev/attachments/20100906/e2b62f0f/attachment.html