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


##########
modules/configuration/src/main/java/org/apache/ignite/internal/configuration/util/ConfigurationUtil.java:
##########
@@ -940,6 +942,80 @@ public T2 next() {
         };
     }
 
+    /**
+     * Removes {@code null} values that correspond to non-deprecated legacy 
keys from the configuration tree.
+     *
+     * @param roots Super root.
+     * @param prefixMap Mutable prefix map with updates received from the 
storage.
+     * @see PublicName#legacyNames()
+     */
+    public static void ignoreLegacyKeys(SuperRoot roots, Map<String, ?> 
prefixMap) {
+        roots.traverseChildren(new KeysTrackingConfigurationVisitor<>() {
+            private Map<String, ?> map = prefixMap;

Review Comment:
   It's not redundant, I use it for recursion. I'll give it a better name



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

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

Reply via email to