timoninmaxim commented on code in PR #11897:
URL: https://github.com/apache/ignite/pull/11897#discussion_r2307047338


##########
modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/snapshot/SnapshotHandlerRestoreTask.java:
##########
@@ -82,36 +98,40 @@ public class SnapshotHandlerRestoreTask extends 
AbstractSnapshotVerificationTask
 
             throw new IgniteException(e);
         }
-
-        return new SnapshotPartitionsVerifyTaskResult(metas, null);
     }
 
     /** Invokes all {@link SnapshotHandlerType#RESTORE} handlers locally. */
-    private static class SnapshotHandlerRestoreJob extends 
AbstractSnapshotVerificationJob {
-        /** Serial version uid. */
-        private static final long serialVersionUID = 0L;
+    private static class SnapshotHandlerRestoreJob {
+        /** */
+        private final IgniteEx ignite;
+
+        /** */
+        private final SnapshotFileTree sft;
+
+        /** */
+        private final Collection<String> rqGrps;
+
+        /** */
+        private final boolean check;
 
         /**
-         * @param snpName Snapshot name.
-         * @param snpPath Snapshot directory path.
-         * @param folderName Folder name for snapshot.
-         * @param consId Consistent id of the related node.
          * @param grps Cache group names.
          * @param check If {@code true} check snapshot before restore.
          */
-        public SnapshotHandlerRestoreJob(
-            String snpName,
-            @Nullable String snpPath,
-            String folderName,
-            String consId,
+        SnapshotHandlerRestoreJob(
+            IgniteEx ignite,
+            SnapshotFileTree sft,
             Collection<String> grps,
             boolean check
         ) {
-            super(snpName, snpPath, folderName, consId, grps, check);
+            this.ignite = ignite;
+            this.sft = sft;
+            this.rqGrps = grps;
+            this.check = check;
         }
 
-        /** {@inheritDoc} */
-        @Override public Map<String, SnapshotHandlerResult<Object>> execute0() 
{
+        /** */
+        public Map<String, SnapshotHandlerResult<Object>> execute0() {
             try {
                 IgniteSnapshotManager snpMgr = 
ignite.context().cache().context().snapshotMgr();
                 SnapshotMetadata meta = 
snpMgr.readSnapshotMetadata(sft.meta());

Review Comment:
   Let's do not touch code out of the ticket scope



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