timoninmaxim commented on a change in pull request #9807:
URL: https://github.com/apache/ignite/pull/9807#discussion_r811142668
##########
File path:
modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/near/consistency/GridNearReadRepairAbstractFuture.java
##########
@@ -58,15 +57,12 @@
/** Maximum number of attempts to remap key to the same primary node. */
protected static final int MAX_REMAP_CNT =
getInteger(IGNITE_NEAR_GET_MAX_REMAPS, DFLT_MAX_REMAP_CNT);
- /** Remap count updater. */
- protected static final
AtomicIntegerFieldUpdater<GridNearReadRepairAbstractFuture> REMAP_CNT_UPD =
-
AtomicIntegerFieldUpdater.newUpdater(GridNearReadRepairAbstractFuture.class,
"remapCnt");
-
- /** Remap count. */
- protected volatile int remapCnt;
+ /** Lsnr calls upd. */
+ private static final
AtomicIntegerFieldUpdater<GridNearReadRepairAbstractFuture> LSNR_CALLS_UPD =
Review comment:
Yes, but
> no many of instances of the future exist
Literally, it's only one per request. Are we going to have many request 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.
To unsubscribe, e-mail: notifications-unsubscr...@ignite.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
