[infinispan-dev] storeAsBinary keeps both the object and the byte[] - why?

[Sanne Grinovero]

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 at redhat.com> wrote:
>
> On 18 Jul 2013, at 14:08, Dan Berindei <dan.berindei at 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 at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/infinispan-dev