[GitHub] [ignite] xtern commented on a change in pull request #9047: IGNITE-14572 Add metastorage to snapshot

[GitBox]

xtern commented on a change in pull request #9047:
URL: https://github.com/apache/ignite/pull/9047#discussion_r623732239



##########
File path: 
modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/snapshot/SnapshotFutureTask.java
##########
@@ -339,7 +348,13 @@ public boolean start() {
                     throw new IgniteCheckedException("Encrypted cache groups 
are not allowed to be snapshot: " + grpId);
 
                 // Create cache group snapshot directory on start in a single 
thread.
-                U.ensureDirectory(cacheWorkDir(tmpConsIdDir, 
cacheDirName(gctx.config())),
+                U.ensureDirectory(cacheWorkDir(tmpConsIdDir, 
FilePageStoreManager.cacheDirName(gctx.config())),
+                    "directory for snapshotting cache group",
+                    log);
+            }
+
+            if (withMetaStorage) {
+                U.ensureDirectory(cacheWorkDir(tmpConsIdDir, 
MetaStorage.METASTORAGE_DIR_NAME),
                     "directory for snapshotting cache group",

Review comment:
       ```suggestion
                       "directory for snapshotting metastorage",
   ```




-- 
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