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

Update instrumentation docs with db client metrics #13080

Merged
merged 5 commits into from
Jan 22, 2025

Conversation

jaydeluca
Copy link
Member

@jaydeluca jaydeluca commented Jan 21, 2025

Related to #12802 #12608 and #594

or should this wait until it is marked stable?

@jaydeluca jaydeluca requested a review from a team as a code owner January 21, 2025 23:43
@github-actions github-actions bot requested a review from theletterf January 21, 2025 23:44
@laurit
Copy link
Contributor

laurit commented Jan 22, 2025

Related to #12802 #12608 and #594

or should this wait until it is marked stable?

I think it is fine to add them. One option would be to add a comment to the footnote that these are not enabled by default of enabled only with the stable semconv opt in.

| [Couchbase Client](https://github.com/couchbase/couchbase-java-client) | 2.0 - 2.7.x,<br>3.1+ | N/A | [Database Client Spans] |
| [Cassandra Driver](https://github.com/datastax/java-driver) | 3.0+ | [opentelemetry-cassandra-4.4](../instrumentation/cassandra/cassandra-4.4/library) | [Database Client Spans], [Database Client Metrics] |
| [Clickhouse Client](https://github.com/ClickHouse/clickhouse-java) | 0.5+ | N/A | [Database Client Spans], [Database Client Metrics] |
| [Couchbase Client](https://github.com/couchbase/couchbase-java-client) | 2.0 - 2.7.x,<br>3.1+ | N/A | [Database Client Spans], [Database Client Metrics] |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

idk if this is worth mentioning here but with couchbase we have 2 instrumentations that generate different telemetry. The one for 3.x that uses tracing built into couchbase does not capture database metrics.

@@ -48,9 +48,9 @@ These are the supported libraries and frameworks:
| [AWS Lambda](https://docs.aws.amazon.com/lambda/latest/dg/java-handler.html) | 1.0+ | [opentelemetry-aws-lambda-core-1.0](../instrumentation/aws-lambda/aws-lambda-core-1.0/library),<br>[opentelemetry-aws-lambda-events-2.2](../instrumentation/aws-lambda/aws-lambda-events-2.2/library) | [FaaS Server Spans] |
| [AWS SDK](https://aws.amazon.com/sdk-for-java/) | 1.11 - 1.12.583,<br>2.2+ | [opentelemetry-aws-sdk-1.11](../instrumentation/aws-sdk/aws-sdk-1.11/library),<br>[opentelemetry-aws-sdk-1.11-autoconfigure](../instrumentation/aws-sdk/aws-sdk-1.11/library-autoconfigure),<br>[opentelemetry-aws-sdk-2.2](../instrumentation/aws-sdk/aws-sdk-2.2/library),<br>[opentelemetry-aws-sdk-2.2-autoconfigure](../instrumentation/aws-sdk/aws-sdk-2.2/library-autoconfigure) | [Messaging Spans], [Database Client Spans], [HTTP Client Spans] |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i think this one also has db client metrics

@trask trask merged commit c16ca44 into open-telemetry:main Jan 22, 2025
58 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants