anton-vinogradov commented on a change in pull request #9814:
URL: https://github.com/apache/ignite/pull/9814#discussion_r816989391
##########
File path:
modules/core/src/test/java/org/apache/ignite/internal/processors/cache/consistency/AbstractReadRepairTest.java
##########
@@ -77,7 +78,7 @@
private static final ConcurrentLinkedDeque<CacheConsistencyViolationEvent>
evtDeq = new ConcurrentLinkedDeque<>();
/** Key. */
- protected static int iterableKey;
+ private static final AtomicInteger iterableKey = new AtomicInteger();
Review comment:
Rarely I was faced with a situation when the next test ignored this
incremental key update made by the previous and fails on the initial entry
update attempt (when it was already inited).
JMM provides no guarantee to sync static files, so I made it thread-safe.
Just decided to fix this at PR where I realized the issue.
--
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]
