[
https://issues.apache.org/jira/browse/IGNITE-2719?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15173341#comment-15173341
]
Semen Boikov commented on IGNITE-2719:
--------------------------------------
Reviewed, looks good, but it seems that GridCacheOnCopyFlagAbstractSelfTest
does not really test p2pEnabled=true since node is started in 'beforeTest'.
Let's merge this fix and then revisit logic for setting 'storeVal' flag in
IgniteCacheObjectProcessorImpl.contextForCache, now it is set when peer class
loading or indexing is enabled, it seems this is not needed when binary
marshaller is used.
> Value is not copied in entry processor if optimized marshaller is used
> ----------------------------------------------------------------------
>
> Key: IGNITE-2719
> URL: https://issues.apache.org/jira/browse/IGNITE-2719
> Project: Ignite
> Issue Type: Bug
> Components: cache
> Reporter: Valentin Kulichenko
> Assignee: Andrey Gura
> Priority: Blocker
> Labels: community, important
> Fix For: 1.6
>
> Attachments: CacheEntryProcessorCopySelfTest.java
>
>
> If {{OptimizedMarshaller}} is used, the {{MutableEntry}} passed to entry
> processor contains the same instance that is stored in cache, even if
> {{copyOnRead}} flag is true.
> This happens because {{CacheLazyEntry.getValue()}} method never creates a
> copy.
> Test attached.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
