(ignite-3) branch main updated: IGNITE-23801 Hocon configuration source should use synthetic key value in logs (#4806)

[apkhmv]

This is an automated email from the ASF dual-hosted git repository.

apkhmv pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/ignite-3.git


The following commit(s) were added to refs/heads/main by this push:
     new 938d43340a IGNITE-23801 Hocon configuration source should use 
synthetic key value in logs (#4806)
938d43340a is described below

commit 938d43340aa80fd692bd6ced37bc01c6ff2841bb
Author: Vadim Pakhnushev <8614891+valep...@users.noreply.github.com>
AuthorDate: Mon Dec 2 00:18:14 2024 +0300

    IGNITE-23801 Hocon configuration source should use synthetic key value in 
logs (#4806)
---
 .../internal/configuration/hocon/HoconListConfigurationSource.java  | 2 +-
 .../ignite/internal/configuration/hocon/HoconConverterTest.java     | 6 ++++++
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git 
a/modules/configuration/src/main/java/org/apache/ignite/internal/configuration/hocon/HoconListConfigurationSource.java
 
b/modules/configuration/src/main/java/org/apache/ignite/internal/configuration/hocon/HoconListConfigurationSource.java
index 28f7eb97b4..bcae565b7c 100644
--- 
a/modules/configuration/src/main/java/org/apache/ignite/internal/configuration/hocon/HoconListConfigurationSource.java
+++ 
b/modules/configuration/src/main/java/org/apache/ignite/internal/configuration/hocon/HoconListConfigurationSource.java
@@ -133,7 +133,7 @@ class HoconListConfigurationSource implements 
ConfigurationSource {
 
             String key = (String) keyValue.unwrapped();
 
-            List<String> path = appendKey(this.path, syntheticKeyName);
+            List<String> path = appendKey(this.path, key);
 
             node.construct(key, new 
HoconObjectConfigurationSource(syntheticKeyName, path, hoconCfg), false);
         }
diff --git 
a/modules/configuration/src/test/java/org/apache/ignite/internal/configuration/hocon/HoconConverterTest.java
 
b/modules/configuration/src/test/java/org/apache/ignite/internal/configuration/hocon/HoconConverterTest.java
index 7960eaeb0a..3e67c28674 100644
--- 
a/modules/configuration/src/test/java/org/apache/ignite/internal/configuration/hocon/HoconConverterTest.java
+++ 
b/modules/configuration/src/test/java/org/apache/ignite/internal/configuration/hocon/HoconConverterTest.java
@@ -699,6 +699,12 @@ public class HoconConverterTest {
                 "rootInjectedName.nested' configuration doesn't have the 
'someName' sub-configuration"
         );
 
+        // Check injected name field value in path
+        assertThrowsIllegalArgException(
+                () -> change("rootInjectedName.nestedNamed = [{someName = foo, 
nonExistingValue = bar}]"),
+                "rootInjectedName.nestedNamed.foo' configuration doesn't have 
the 'nonExistingValue' sub-configuration"
+        );
+
         assertThrowsIllegalArgException(
                 () -> asHoconStr(List.of("rootInjectedName"), "nested", 
"someName"),
                 "Configuration value 'rootInjectedName.nested.someName' has 
not been found"