tkalkirill commented on code in PR #2772:
URL: https://github.com/apache/ignite-3/pull/2772#discussion_r1392222766


##########
modules/table/src/main/java/org/apache/ignite/internal/table/distributed/command/PartitionCommand.java:
##########
@@ -38,5 +41,15 @@ public interface PartitionCommand extends 
SafeTimePropagatingCommand, CatalogVer
      * Returns version that the Catalog must have locally for the node to be 
allowed to accept this command via replication.
      */
     @Override
+    @Transient
+    @WithSetter
     int requiredCatalogVersion();
+
+    /**
+     * Setter for {@link #requiredCatalogVersion()}. Called by the creator or 
the {@link PartitionCommandsMarshaller} while deserializing.
+     */
+    @Override
+    default void requiredCatalogVersion(int version) {

Review Comment:
   Yes, but I see that you will try to correct the comment above, perhaps the 
question will disappear then.



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