tkalkirill commented on code in PR #902:
URL: https://github.com/apache/ignite-3/pull/902#discussion_r910672841
##########
modules/page-memory/src/main/java/org/apache/ignite/internal/pagememory/persistence/PersistentPageMemory.java:
##########
@@ -1476,11 +1476,6 @@ private void resetDirtyPages() {
public boolean tryToRemovePage(FullPageId fullPageId, long absPtr)
throws IgniteInternalCheckedException {
assert writeLock().isHeldByCurrentThread();
- // Do not evict group meta pages.
Review Comment:
Because it was stored in the index partition, and now it will not be. It is
not entirely clear whether this page will be needed in the future, in 2.0
information for snapshots was stored there before metastorage, now I think this
information will be located in the configuration.
--
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]
