You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@ignite.apache.org by GitBox <gi...@apache.org> on 2021/08/06 15:33:54 UTC

[GitHub] [ignite-3] ibessonov commented on a change in pull request #262: IGNITE-15264 Fixed wrong applied revision handling in DistributedConfigurationStorage

ibessonov commented on a change in pull request #262:
URL: https://github.com/apache/ignite-3/pull/262#discussion_r684326770



##########
File path: modules/runner/src/main/java/org/apache/ignite/internal/storage/DistributedConfigurationStorage.java
##########
@@ -83,7 +84,7 @@
     private volatile ConfigurationStorageListener lsnr;
 
     /** Storage version. It stores actual meta storage revision, that is applied to configuration manager. */
-    private final AtomicLong ver = new AtomicLong(0L);
+    private final AtomicLong changeId = new AtomicLong(0L);

Review comment:
       Well, technically that is still revision of the local metastorage at the moment of last configuration update (including node start). Maybe I should clarify that in the comment?




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