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 <[email protected]> wrote: > > On 18 Jul 2013, at 14:08, Dan Berindei <[email protected]> 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 > [email protected] > https://lists.jboss.org/mailman/listinfo/infinispan-dev _______________________________________________ infinispan-dev mailing list [email protected] https://lists.jboss.org/mailman/listinfo/infinispan-dev
