-
Notifications
You must be signed in to change notification settings - Fork 150
Add messaging wrappers to support lightweight manual instrumentation #1803
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
Conversation
Several suggestions from trask:
Thanks for @trask ! |
@trask Tests for mns-clients and api itself are not implemented in view of possible migration from Looking forward to your response and have nice days on KubeCon EU! |
BTW, please ignore the build failure caused by mns. Our sdk of 1.3.0 hasn't been released yet and we will release it soon. |
```xml | ||
<dependency> | ||
<groupId>io.opentelemetry.contrib</groupId> | ||
<artifactId>opentelemetry-messaging-wrappers-api</artifactId> |
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.
Add following version like other modules? https://github.com/open-telemetry/opentelemetry-java-contrib/tree/main/inferred-spans#usage
<version>{latest version}</version>
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.
Please see #1855 for newest version.
Description:
< Describe what is being changed or added.
Ex. Bug fix - Describe the bug and how this fixes it.
Ex. Feature addition - Describe what this provides and why. >
Existing Issue(s):
< Link any applicable issues. >
Testing:
< Describe what testing was performed and any tests were added. >
Documentation:
< Describe the documentation added.
Please be sure to modify relevant existing documentation if needed. >
Outstanding items:
< Anything that these changes are intentionally missing
that will be needed or can be helpful in the future. >