Mmuzaf commented on a change in pull request #6951: Ignite 11073 12069 P2P Rebalance collaboration work URL: https://github.com/apache/ignite/pull/6951#discussion_r407631903
########## File path: modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/GridCacheOffheapManager.java ########## @@ -1625,6 +1629,12 @@ private DataEntryRow(DataEntry entry) { /** */ private final CountDownLatch latch = new CountDownLatch(1); + /** Currently used data storage state. */ + private final AtomicBoolean active = new AtomicBoolean(true); Review comment: If storage is not `active` but, for instance, `fullSize` method accessed should we throw an exception in this case? This is about some guarantees which no one accesses to cachedatastore while the partition file is swapped. ---------------------------------------------------------------- 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