adapted from feedback

Author: jackshirazi

api/incubator/src/main/java/io/opentelemetry/api/incubator/config/ConfigProvider.java

@@ -99,6 +99,8 @@ default ConfigChangeRegistration addConfigChangeListener(
    *
    * @param path the declarative configuration path to update
    * @param newSubtree the new configuration subtree to set at the path
+   * @throws DeclarativeConfigException if an intermediate segment of the path resolves to a
+   *     non-mapping value (e.g., a scalar or list), indicating a schema conflict
    */
   default void updateConfig(String path, DeclarativeConfigProperties newSubtree) {}
 
@@ -116,6 +118,8 @@ default void updateConfig(String path, DeclarativeConfigProperties newSubtree) {
    * @param key the property key within the path
    * @param value the new value for the property (must be a scalar: String, Boolean, Long, Double,
    *     Integer, or a List of scalars)
+   * @throws DeclarativeConfigException if an intermediate segment of the path resolves to a
+   *     non-mapping value (e.g., a scalar or list), indicating a schema conflict
    */
   default void setConfigProperty(String path, String key, Object value) {}
 

api/incubator/src/main/java/io/opentelemetry/api/incubator/config/DeclarativeConfigProperties.java

@@ -43,22 +43,6 @@ static Map<String, Object> toMap(DeclarativeConfigProperties declarativeConfigPr
     return DeclarativeConfigPropertyUtil.toMap(declarativeConfigProperties);
   }
 
-  /**
-   * Create a {@link DeclarativeConfigProperties} from a {@code Map<String, Object>}.
-   *
-   * <p>This is the inverse of {@link #toMap(DeclarativeConfigProperties)}. Values in the map are
-   * expected to follow the same conventions: scalars, lists of scalars, nested maps, and lists of
-   * maps.
-   *
-   * @param map the map to wrap
-   * @param componentLoader the component loader to use
-   * @return a {@link DeclarativeConfigProperties} backed by the map
-   */
-  static DeclarativeConfigProperties fromMap(
-      Map<String, Object> map, ComponentLoader componentLoader) {
-    return new MapBackedDeclarativeConfigProperties(map, componentLoader);
-  }
-
   /**
    * Returns a {@link String} configuration property.
    *

api/incubator/src/main/java/io/opentelemetry/api/incubator/config/MapBackedDeclarativeConfigProperties.java

@@ -9,7 +9,7 @@
 
 import com.google.common.collect.ImmutableMap;
 import io.opentelemetry.sdk.extension.incubator.fileconfig.DeclarativeConfiguration;
-import io.opentelemetry.sdk.extension.incubator.fileconfig.YamlDeclarativeConfigProperties;
+import io.opentelemetry.sdk.internal.YamlDeclarativeConfigProperties;
 import io.opentelemetry.sdk.internal.SdkConfigProvider;
 import java.io.ByteArrayInputStream;
 import java.nio.charset.StandardCharsets;