[infinispan-issues] [JBoss JIRA] (ISPN-4441) Improvements on getCacheEntry() synchronization

[Galder Zamarreño (JIRA)]

     [ https://issues.jboss.org/browse/ISPN-4441?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Galder Zamarreño resolved ISPN-4441.
------------------------------------

    Fix Version/s:     (was: 7.0.0.Alpha5)
       Resolution: Won't Fix


The final solution used clone() which solves the issue pointed out by Pedro. Separating get commands is not urgent.

> Improvements on getCacheEntry() synchronization
> -----------------------------------------------
>
>                 Key: ISPN-4441
>                 URL: https://issues.jboss.org/browse/ISPN-4441
>             Project: Infinispan
>          Issue Type: Enhancement
>      Security Level: Public(Everyone can see) 
>          Components: Core
>            Reporter: Galder Zamarreño
>            Assignee: Galder Zamarreño
>
> - Separate GetKeyValueCommand into two, GetKeyValueCommand and GetCacheEntryCommand to have good separation and avoid conditional logic in GetKeyValueCommand.
> - As indicated by Pedro in https://github.com/infinispan/infinispan/pull/2671:
> {quote}
> I had something simpler in mind. you could invoke create(cacheEntry) that returns a mutable copy. I think it is ok the copy to be mutable, but it you really want it immutable, you can do immutableInternalCacheEntry(create(cacheEntry))
> this way you would avoid all the changes in CoreImmutables
> {quote}
> - Finally, sort out TODO in documentation of InternalEntryFactory



--
This message was sent by Atlassian JIRA
(v6.2.6#6264)