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 JDBC tracing info to the OpenTelemetry guide #24583

Merged
merged 1 commit into from
Mar 28, 2022

Conversation

jhalliday
Copy link
Contributor

OpenTelemetry version of the older OpenTracing JDBC setup steps

@quarkus-bot
Copy link

quarkus-bot bot commented Mar 28, 2022

Thanks for your pull request!

The title of your pull request does not follow our editorial rules. Could you have a look?

  • title should not end up with dot

This message is automatically generated by a bot.

@jhalliday jhalliday changed the title Add JDBC tracing info to the OpenTelemetry guide. Add JDBC tracing info to the OpenTelemetry guide Mar 28, 2022
implementation("io.opentelemetry.instrumentation:opentelemetry-jdbc")
----

As it uses a dedicated JDBC driver, you must configure your datasource and Hibernate to use it.
Copy link
Member

Choose a reason for hiding this comment

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

Looks good, thanks!
I will just s/Hibernate/Hibernate ORM/ and merge.

@gsmet gsmet added triage/waiting-for-ci Ready to merge when CI successfully finishes triage/backport? labels Mar 28, 2022
@gsmet gsmet merged commit 4f5dfa5 into quarkusio:main Mar 28, 2022
@quarkus-bot quarkus-bot bot added this to the 2.9 - main milestone Mar 28, 2022
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label Mar 28, 2022
@romilpunetha
Copy link

Does something like this exist for Mongodb?

@gsmet gsmet modified the milestones: 2.9 - main, 2.8.0.Final Mar 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants