]
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