Skip to content

Commit 408174d

Browse files
committed
pr review
1 parent de5c5f6 commit 408174d

File tree

2 files changed

+9
-0
lines changed

2 files changed

+9
-0
lines changed

instrumentation/spring/spring-boot-autoconfigure/src/main/java/io/opentelemetry/instrumentation/spring/autoconfigure/SpringConfigProvider.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,11 @@
1717
/**
1818
* Spring flavor of {@link io.opentelemetry.sdk.extension.incubator.fileconfig.SdkConfigProvider}
1919
* that tries to coerce types, because spring doesn't tell what the original type was.
20+
*
21+
* <p>The entire class is a copy of <a
22+
* href="https://github.com/open-telemetry/opentelemetry-java/blob/main/sdk-extensions/incubator/src/main/java/io/opentelemetry/sdk/extension/incubator/fileconfig/SdkConfigProvider.java">SdkConfigProvider</a>
23+
* which uses {@link SpringDeclarativeConfigProperties} instead of {@link
24+
* io.opentelemetry.sdk.extension.incubator.fileconfig.YamlDeclarativeConfigProperties}.
2025
*/
2126
final class SpringConfigProvider implements ConfigProvider {
2227

instrumentation/spring/spring-boot-autoconfigure/src/main/java/io/opentelemetry/instrumentation/spring/autoconfigure/SpringDeclarativeConfigProperties.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,10 @@
2727
* Spring flavor of {@link
2828
* io.opentelemetry.sdk.extension.incubator.fileconfig.YamlDeclarativeConfigProperties}, that tries
2929
* to coerce types, because spring doesn't tell what the original type was.
30+
*
31+
* <p>The entire class is a copy of <a
32+
* href="https://github.com/open-telemetry/opentelemetry-java/blob/main/sdk-extensions/incubator/src/main/java/io/opentelemetry/sdk/extension/incubator/fileconfig/YamlDeclarativeConfigProperties.java">YamlDeclarativeConfigProperties</a>
33+
* with only minor modifications to type coercion logic.
3034
*/
3135
final class SpringDeclarativeConfigProperties implements DeclarativeConfigProperties {
3236

0 commit comments

Comments
 (0)