anton-vinogradov commented on a change in pull request #9814:
URL: https://github.com/apache/ignite/pull/9814#discussion_r816985263
##########
File path:
modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/near/GridNearTxLocal.java
##########
@@ -2500,7 +2500,7 @@ private boolean enlistWriteEntry(GridCacheContext
cacheCtx,
keepCacheObjects,
deserializeBinary,
false,
- getRes,
+ null,
Review comment:
The previous code was wrong but never caused the problem before the
change.
Passing the getRes we'll return it to the user and in the getEntry case, the
binary value will be deserialized.
But we still want to expect CacheObject at futures post-closure listeners
(like on tx commit), but may gain deserialized Integer instead.
So, this just a copy-on-write fix allows us to have futures data unchanged
at its listeners.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
