New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move rest of non-specd attributes to experimental #2402
Move rest of non-specd attributes to experimental #2402
Conversation
private static final boolean CAPTURE_EXPERIMENTAL_SPAN_ATTRIBUTES = | ||
Config.get() | ||
.getBooleanProperty( | ||
"otel.instrumentation.apache-camel.experimental-span-attributes", false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm, can you use the constant from BaseSpanDecorator
if you make it protected?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 will make this change (in this PR) tomorrow
@trask what's the problem with calling |
A couple possible solutions would be to init the default config with system properties for library instrumentation users, or to have an instrumentation-specific configuration API that library users (and the javaagent) can use to configure it. |
Almost completes #1874, except for aws-sdk-2.2, because we don't have solution for
Config.get()
in library instrumentation yet.