Also the implementation of the new "defensive copy" feature is not
correct. The reference to the unmarshalled instance is cached in a
field of MarshalledValue and there are no barriers preventing multiple
GET operations to retrieve the same instance.
Should we reopen the issue or create a new one?
On 18 July 2013 14:48, Mircea Markus <mmarkus(a)redhat.com> wrote:
On 18 Jul 2013, at 14:08, Dan Berindei <dan.berindei(a)gmail.com> wrote:
> Are you sure both representations are kept?
>
> MarshalledValueInterceptor.visitGetKeyValueCommand() should call
MarshalledValue.compact just before returning, which should keep only one representation.
MarshalledValueInterceptor.visitGetKeyValueCommand() invokes
MarshalledValueInterceptor.compact(mv). The *mv* parameter , at least for values, is
null.
Cheers,
--
Mircea Markus
Infinispan lead (
www.infinispan.org)
_______________________________________________
infinispan-dev mailing list
infinispan-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/infinispan-dev