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


##########
modules/table/src/main/java/org/apache/ignite/internal/table/distributed/command/CatalogVersionAware.java:
##########
@@ -25,4 +25,9 @@ public interface CatalogVersionAware {
      * Returns version that the Catalog must have locally for the node to be 
allowed to accept this command via replication.
      */
     int requiredCatalogVersion();
+
+    /**
+     * Setter for the required catalog version. Should be used with caution. 
Only object's creator should call it.
+     */
+    void requiredCatalogVersion(int version);

Review Comment:
   I change my opinion, this is the right place for this method. 
PartitionCommand might not be the only sub-interface of CatalogCersionAware in 
the future



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