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 support for OpenTelemetry in JDBC connectors #18735

Merged
merged 2 commits into from
Aug 21, 2023

Conversation

ebyhr
Copy link
Member

@ebyhr ebyhr commented Aug 19, 2023

Description

Support OpenTelemetry in JDBC connector. Attached the screenshot of SQL Server's example:
Screenshot 2023-08-19 at 9 43 14

Release notes

(x) This is not user-visible or is docs only, and no release notes are required.

@cla-bot cla-bot bot added the cla-signed label Aug 19, 2023
@ebyhr ebyhr force-pushed the ebi/jdbc-opentelemetry branch 2 times, most recently from a155aaa to 3eb5e48 Compare August 19, 2023 02:24
@ebyhr ebyhr force-pushed the ebi/jdbc-opentelemetry branch from 3eb5e48 to 8ab3eab Compare August 20, 2023 23:21
Copy link
Contributor

@wendigo wendigo left a comment

Choose a reason for hiding this comment

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

LGTM % version question

plugin/trino-base-jdbc/pom.xml Outdated Show resolved Hide resolved
@ebyhr ebyhr force-pushed the ebi/jdbc-opentelemetry branch from 8ab3eab to 3c09177 Compare August 21, 2023 05:30
@ebyhr ebyhr merged commit 7b269fa into trinodb:master Aug 21, 2023
@ebyhr ebyhr deleted the ebi/jdbc-opentelemetry branch August 21, 2023 06:55
@github-actions github-actions bot added this to the 425 milestone Aug 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants