AMashenkov commented on a change in pull request #5995: IGNITE-10261 MVCC: cache operation may hang during late affinity assignment. URL: https://github.com/apache/ignite/pull/5995#discussion_r262559666
########## File path: modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtGetSingleFuture.java ########## @@ -311,7 +313,14 @@ private boolean map(KeyCacheObject key, boolean forceKeys) { // By reserving, we make sure that partition won't be unloaded while processed. if (part.reserve()) { - if (forceKeys || (part.state() == OWNING || part.state() == LOST)) { + boolean mvccNoOwners = false; + + // Force key request is disabled for MVCC. So if there are no partition owners for the given key + // we need to set flag forceKeys to true to avoid useless remapping to other non-owning partitions. + if (cctx.mvccEnabled()) Review comment: Should partition state be LOST if it has no owners? Is additional flag really needed here? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services