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