Re: [infinispan-dev] Fixing ISPN-2384 (data loss with concurrent activation/passivation)

2012-11-06 Thread Galder Zamarreño
On Nov 2, 2012, at 7:08 PM, Dennis Reed der...@redhat.com wrote: On 11/02/2012 12:45 PM, Galder Zamarreño wrote: Ok, my suggestion below is working now, but I'm not so sure the way the activation interceptor works is the correct one. IMO, an activation, which involves removing an entry

Re: [infinispan-dev] Fixing ISPN-2384 (data loss with concurrent activation/passivation)

2012-11-02 Thread Galder Zamarreño
Ok, my suggestion below is working now, but I'm not so sure the way the activation interceptor works is the correct one. IMO, an activation, which involves removing an entry from the cache store (and updating the stats), should only be done when a cache entry has been retrieved from the cache

Re: [infinispan-dev] Fixing ISPN-2384 (data loss with concurrent activation/passivation)

2012-11-02 Thread Dennis Reed
On 11/02/2012 12:45 PM, Galder Zamarreño wrote: Ok, my suggestion below is working now, but I'm not so sure the way the activation interceptor works is the correct one. IMO, an activation, which involves removing an entry from the cache store (and updating the stats), should only be done

Re: [infinispan-dev] Fixing ISPN-2384 (data loss with concurrent activation/passivation)

2012-10-31 Thread Galder Zamarreño
On Oct 30, 2012, at 6:23 PM, Galder Zamarreño gal...@redhat.com wrote: Hi all, Re: https://issues.jboss.org/browse/ISPN-2384 I've created a unit test in https://github.com/galderz/infinispan/commit/01230d40df6f26720039986916c38de8be33b44b That was the easy part :). How to fix it is

[infinispan-dev] Fixing ISPN-2384 (data loss with concurrent activation/passivation)

2012-10-30 Thread Galder Zamarreño
Hi all, Re: https://issues.jboss.org/browse/ISPN-2384 I've created a unit test in https://github.com/galderz/infinispan/commit/01230d40df6f26720039986916c38de8be33b44b That was the easy part :). How to fix it is no so clear. In pseudo-code, the race condition happens when: 1. T1. passivate