• 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:

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

• Finally, sort out TODO in documentation of InternalEntryFactory