Rename otel.instrumentation.experimental.span-suppression-strategy to otel.instrumentation.common.experimental.span-suppression-strategy to follow common naming conventions.
This will allow us to clean up
|
SpanSuppressor buildSpanSuppressor() { |
|
// otel.instrumentation.experimental.* doesn't fit the usual pattern of configuration properties |
|
// for instrumentations, so we need to handle both declarative and non-declarative configs here |
|
String value = |
|
ConfigPropertiesUtil.isDeclarativeConfig(openTelemetry) |
|
? ConfigPropertiesUtil.getString( |
|
openTelemetry, "common", "span_suppression_strategy/development") |
|
.orElse(null) |
|
: ConfigPropertiesUtil.getString( |
|
"otel.instrumentation.experimental.span-suppression-strategy"); |
|
return new SpanSuppressors.ByContextKey( |
|
SpanSuppressionStrategy.fromConfig(value).create(getSpanKeysFromAttributesExtractors())); |
|
} |
Rename
otel.instrumentation.experimental.span-suppression-strategytootel.instrumentation.common.experimental.span-suppression-strategyto follow common naming conventions.This will allow us to clean up
opentelemetry-java-instrumentation/instrumentation-api/src/main/java/io/opentelemetry/instrumentation/api/instrumenter/InstrumenterBuilder.java
Lines 370 to 382 in c9d0e19