[infinispan-issues] [JBoss JIRA] Commented: (ISPN-912) Transactional replace calls maybe not propagated to cache store

[Galder Zamarreño (JIRA)]

    [ https://issues.jboss.org/browse/ISPN-912?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12580108#comment-12580108 ] 

Galder Zamarreño commented on ISPN-912:
---------------------------------------

The following test shows the issue:
{code}
public void testTransactionalReplace(Method m) throws Exception {
   assert cache.getStatus() == ComponentStatus.RUNNING;
   assertNotInCacheAndStore(k(m, 1));
   assertNotInCacheAndStore(k(m, 2));

   cache.put(k(m, 2), v(m));

   tm.begin();
   cache.put(k(m, 1), v(m, 1));
   cache.replace(k(m, 2), v(m, 1));
   Transaction t = tm.suspend();

   assertNotInCacheAndStore(k(m, 1));

   tm.resume(t);
   tm.commit();

   assertInCacheAndStore(k(m, 1), v(m, 1));
   assertInCacheAndStore(k(m, 2), v(m, 1));
}
{code}

> Transactional replace calls maybe not propagated to cache store
> ---------------------------------------------------------------
>
>                 Key: ISPN-912
>                 URL: https://issues.jboss.org/browse/ISPN-912
>             Project: Infinispan
>          Issue Type: Bug
>          Components: Loaders and Stores
>    Affects Versions: 4.2.0.Final, 4.2.1.CR1, 5.0.0.ALPHA2
>            Reporter: Galder Zamarreño
>            Assignee: Galder Zamarreño
>             Fix For: 4.2.1.Final, 5.0.0.ALPHA3
>
>
> Why doesn't StoreModificationsBuilder in CacheStoreInterceptor override visitReplaceCommand()?
> A replace is a modification which should be converted into a Store() command when executed within a transaction.
> Should be easy to create a unit test to verify whether this is working, or check the testsuite for one.

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira