You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ignite.apache.org by "alievmirza (via GitHub)" <gi...@apache.org> on 2025/12/22 18:17:26 UTC

Re: [PR] IGNITE-27142 Remove non-clocated code (disaster recovery) [ignite-3]

alievmirza commented on code in PR #7265:
URL: https://github.com/apache/ignite-3/pull/7265#discussion_r2640774599


##########
modules/table/src/main/java/org/apache/ignite/internal/table/distributed/disaster/GroupUpdateRequest.java:
##########
@@ -34,22 +34,14 @@
  * the original group, in the case of the automatic reset.
  */
 class GroupUpdateRequest implements DisasterRecoveryRequest {
-
     private final UUID operationId;
 
-    /**
-     * Catalog version at the moment of operation creation. Must match catalog version at the moment of operation execution.
-     */
+    /** Catalog version at the moment of operation creation. Must match catalog version at the moment of operation execution. */
     private final int catalogVersion;
 
     private final int zoneId;
 
-    private final boolean colocationEnabled;
-
-    /**
-     * Map of (tableId -> setOfPartitions) to reset if colocation is disabled
-     * or map of (zoneId -> setOfPartitions) to reset if colocation is enabled.
-     */
+    /** Map of (zoneId -> setOfPartitions) to reset if colocation is enabled. */

Review Comment:
   I think that we don't need `if colocation is enabled` part



-- 
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-unsubscribe@ignite.apache.org

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