open-telemetry
Make IncludeExcludePredicate public
Is your feature request related to a problem? Please describe.
https://github.com/open-telemetry/opentelemetry-java-contrib/pull/2031 makes use of IncludeExcludePredicate
Describe the solution you'd like
make the class public
Part of https://github.com/open-telemetry/opentelemetry-java/issues/6970
I think we would probably not want to make this a part of our official SDK API surface area. It's an internal utility class, and that's not really the business of the OpenTelemetry SDK to be publishing.
maybe not in this form, but I could see an argument for declarative config supporting this in some way that doesn't require copy-pasting this code everywhere (https://github.com/open-telemetry/opentelemetry-configuration/blob/main/CONTRIBUTING.md#properties-which-pattern-matching)
maybe not in this form, but I could see an argument for declarative config supporting this in some way that doesn't require copy-pasting this code everywhere (https://github.com/open-telemetry/opentelemetry-configuration/blob/main/CONTRIBUTING.md#properties-which-pattern-matching)
I could see that as a possibility, if @jack-berg is ok supporting it forever as a part of the declarative config APIs.
We could even add a sentence to the spec:
SDKs SHOULD provide a method named ... to parse this