Skip to content
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

Add rpc.system value for Apache Dubbo #2453

Merged

Conversation

trask
Copy link
Member

@trask trask commented Mar 29, 2022

Changes

Adds rpc.system value for Apache Dubbo.

We have Java instrumentation for Apache Dubbo, and would like to "upstream" an rpc.system value for it.

Btw, suggesting a value of apache_dubbo over just dubbo, because as @tydhot mentioned in open-telemetry/opentelemetry-java-instrumentation#5432:

At present, many users' old systems in China are using Alibaba Dubbo. RPC types here need to be distinguished, otherwise there will be some confusion in observation.

@trask trask force-pushed the add-apache-dubbo-rpc-system-value branch from 64dab19 to 3e82906 Compare March 29, 2022 04:42
@trask trask marked this pull request as ready for review March 29, 2022 04:42
@trask trask requested review from a team March 29, 2022 04:42
@trask trask requested review from a team March 29, 2022 04:46
@arminru arminru added area:semantic-conventions Related to semantic conventions spec:trace Related to the specification/trace directory spec:metrics Related to the specification/metrics directory labels Mar 29, 2022
@bogdandrutu bogdandrutu merged commit fd9be6f into open-telemetry:main Mar 30, 2022
@trask trask deleted the add-apache-dubbo-rpc-system-value branch March 30, 2022 22:34
carlosalberto pushed a commit to carlosalberto/opentelemetry-specification that referenced this pull request Oct 31, 2024
* Add rpc.system value for Apache Dubbo

* Generate table
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:semantic-conventions Related to semantic conventions spec:metrics Related to the specification/metrics directory spec:trace Related to the specification/trace directory
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants