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 2022/02/02 08:19:09 UTC

[GitHub] [ignite-3] ibessonov commented on a change in pull request #611: IGNITE-16434 @InternalId integration for table id.

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



##########
File path: modules/configuration/src/main/java/org/apache/ignite/internal/configuration/tree/NamedListNode.java
##########
@@ -489,9 +489,26 @@ public void construct(String key, ConfigurationSource src, boolean includeIntern
 
         newElement.setInjectedNameFieldValue(key);
 
+        newElement.internalId(generateInternalId());
+
         return new ElementDescriptor(newElement);
     }
 
+    /**
+     * Generates a new unique internal id for the node.
+     *
+     * @return Generated internal id.
+     */
+    private UUID generateInternalId() {

Review comment:
       It will be re-generated for every retry attempt




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