Right, so basically it's time for me to integrate this into HB trunk. I'll start with that work asap. On 08/18/2009 08:36 PM, Steve Ebersole wrote: > To clarify further... > > Within txn here there are calls to: > 1) cacheAccess.lockRegion() > 2) cacheAccess.removeAll() > > In after-completion phase there is a call to: > 3) cacheAccess.unlockRegion(lock-from-#1) > > A transactional cache would not care about #1 nor #3... > > > On Tue, 2009-08-18 at 13:25 -0500, Steve Ebersole wrote: >> On Tue, 2009-08-18 at 16:50 +0200, Galder Zamarreno wrote: >>> This change of behaivour is making Infinispan cache provider >>> tests that >>> do bulk modifications to fail. The reason it fails is because >>> Hibernate >>> has a javax.transaction.Synchronization implementation called >>> CacheSynchronization that in it's afterCompletion(), it leads to >>> call >>> BulkOperationCleanupAction.evictEntityRegions() which clears the >>> cache >>> for the affected entities. Now, since the tx status is COMMITTED, >>> the >>> test fails. >> This is no longer accurate. There was a bug in how >> BulkOperationCleanupAction worked because it was still using the >> older >> Hibrnate cache SPIs. See >> http://opensource.atlassian.com/projects/hibernate/browse/HHH-4034 >> >> -- Galder Zamarreño Sr. Software Engineer Infinispan, JBoss Cache _______________________________________________ infinispan-dev mailing list infinispan-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev

A FYI: Integrated with HB trunk and test now passes without any probs. Many thanks Steve! On 08/18/2009 10:57 PM, Manik Surtani wrote: > Great! > > On 18 Aug 2009, at 20:40, Galder Zamarreno wrote: > >> Right, so basically it's time for me to integrate this into HB >> trunk. >> I'll start with that work asap. >> >> On 08/18/2009 08:36 PM, Steve Ebersole wrote: >>> To clarify further... >>> >>> Within txn here there are calls to: >>> 1) cacheAccess.lockRegion() >>> 2) cacheAccess.removeAll() >>> >>> In after-completion phase there is a call to: >>> 3) cacheAccess.unlockRegion(lock-from-#1) >>> >>> A transactional cache would not care about #1 nor #3... >>> >>> >>> On Tue, 2009-08-18 at 13:25 -0500, Steve Ebersole wrote: >>>> On Tue, 2009-08-18 at 16:50 +0200, Galder Zamarreno wrote: >>>>> This change of behaivour is making Infinispan cache provider >>>>> tests >>>>> that >>>>> do bulk modifications to fail. The reason it fails is because >>>>> Hibernate >>>>> has a javax.transaction.Synchronization implementation called >>>>> CacheSynchronization that in it's afterCompletion(), it leads >>>>> to call >>>>> BulkOperationCleanupAction.evictEntityRegions() which clears >>>>> the cache >>>>> for the affected entities. Now, since the tx status is >>>>> COMMITTED, the >>>>> test fails. >>>> This is no longer accurate. There was a bug in how >>>> BulkOperationCleanupAction worked because it was still using the >>>> older >>>> Hibrnate cache SPIs. See >>>> http://opensource.atlassian.com/projects/hibernate/browse/HHH-4034 >>>> >>>> >> >> -- >> Galder Zamarreño >> Sr. Software Engineer >> Infinispan, JBoss Cache >> _______________________________________________ >> infinispan-dev mailing list >> infinispan-dev(a)lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/infinispan-dev > > -- > Manik Surtani > manik(a)jboss.org > Lead, Infinispan > Lead, JBoss Cache > http://www.infinispan.org > http://www.jbosscache.org > > > > -- Galder Zamarreño Sr. Software Engineer Infinispan, JBoss Cache