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.

Sign up for GitHub

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

Jump to bottom

dbclient: remove dependency on opentracing-api #7617

Closed
barchetta opened this issue Sep 18, 2023 · 0 comments · Fixed by #7709
Closed

dbclient: remove dependency on opentracing-api #7617

barchetta opened this issue Sep 18, 2023 · 0 comments · Fixed by #7709
Assignees
@dalexandrov @Tomas-Kraus
Labels
3.x Issues for 3.x version branch 4.x Version 4.x DB client Helidon DB Client
Milestone
4.0.0

Comments

@barchetta
Copy link
Member

dbclient/tracing has a dependency on io.opentracing:opentracing-api. It should not. Code should be updated to use the Helidon SE tracing API.

This should be fixed in 4.x and 3.x.
@barchetta barchetta added DB client Helidon DB Client 3.x Issues for 3.x version branch 4.x Version 4.x labels Sep 18, 2023
@barchetta barchetta added this to the 4.0.0-RC1 milestone Sep 18, 2023
@barchetta barchetta mentioned this issue Sep 18, 2023
Closed
12 tasks
@barchetta barchetta modified the milestones: 4.0.0-RC1, 4.0.0-RC2 Sep 25, 2023
This was referenced Oct 3, 2023
Merged
Merged
@barchetta barchetta modified the milestones: 4.0.0-RC2, 4.0.0 Oct 3, 2023
@m0mus m0mus added this to Backlog Aug 12, 2024
@m0mus m0mus moved this to Closed in Backlog Aug 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dalexandrov dalexandrov

@Tomas-Kraus Tomas-Kraus

Labels
3.x Issues for 3.x version branch 4.x Version 4.x DB client Helidon DB Client
Projects
Backlog
Archived in project
Milestone
4.0.0
Development

Successfully merging a pull request may close this issue.

[3.x] - Migrate opentracing to Helidon Tracing dalexandrov/helidon
3 participants
@dalexandrov @barchetta @Tomas-Kraus