sashapolo commented on code in PR #5443:
URL: https://github.com/apache/ignite-3/pull/5443#discussion_r2005074766


##########
modules/storage-api/src/main/java/org/apache/ignite/internal/storage/engine/MvPartitionMeta.java:
##########
@@ -26,20 +27,50 @@
 public class MvPartitionMeta extends PrimitivePartitionMeta {
     private final byte[] groupConfig;
 
+    private final byte[] snapshotInfo;
+
     /** Constructor. */
     public MvPartitionMeta(
             long lastAppliedIndex,
             long lastAppliedTerm,
             byte[] groupConfig,
-            @Nullable LeaseInfo leaseInfo
+            @Nullable LeaseInfo leaseInfo,
+            byte[] snapshotInfo
     ) {
         super(lastAppliedIndex, lastAppliedTerm, leaseInfo);
 
         this.groupConfig = groupConfig;
+        this.snapshotInfo = snapshotInfo;

Review Comment:
   In this PR, objects of this class are created on-the-fly, they are not 
persisted anywhere, so they can't be null. Also, this PR is only needed for the 
colocation case, which is disabled on older versions



-- 
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: notifications-unsubscr...@ignite.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to