Vladsz83 commented on code in PR #11495:
URL: https://github.com/apache/ignite/pull/11495#discussion_r1764901121


##########
modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/snapshot/IgniteSnapshotManager.java:
##########
@@ -1987,7 +1984,7 @@ public List<File> snapshotCacheDirectories(String 
snpName, @Nullable String snpP
      * @return Snapshot metadata instance.
      */
     public SnapshotMetadata readSnapshotMetadata(File snpDir, String consId) 
throws IgniteCheckedException, IOException {
-        return readSnapshotMetadata(new File(snpDir, 
snapshotMetaFileName(consId)));
+        return snpChecker.readSnapshotMetadata(snpDir, consId);

Review Comment:
   We should keep the ability to read and validate metas, partitions 
independently of 'heavy' kernel and managers for external tools. Maybe we need 
a snapshot file manager.



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

Reply via email to