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

UCP integration needs to depend on ucp11, not ucp #9349

Open
ljnelson opened this issue Oct 9, 2024 · 0 comments · May be fixed by #9353
Open

UCP integration needs to depend on ucp11, not ucp #9349

ljnelson opened this issue Oct 9, 2024 · 0 comments · May be fixed by #9353
Assignees
@ljnelson
Labels
4.x Version 4.x bug Something isn't working jpa/jta

Comments

@ljnelson
Copy link
Member

ljnelson commented Oct 9, 2024

Because of ef6649f, the Persistence Guide now makes reference to our ojdbc integration. That integration brings in the version of UCP that was compiled with JDK 11 (ucp11). The actual UCP CDI integration brings in the version of UCP that was compiled with JDK 8 (ucp), which is still supported. Nevertheless, the UCP CDI integration should include ucp11, not ucp.

Without a fix, users will end up with two copies of UCP in their classpath (ucp and ucp11) at runtime.
@ljnelson ljnelson added bug Something isn't working jpa/jta 4.x Version 4.x labels Oct 9, 2024
@ljnelson ljnelson self-assigned this Oct 9, 2024
@ljnelson ljnelson added this to Backlog Oct 9, 2024
@github-project-automation github-project-automation bot moved this to Triage in Backlog Oct 9, 2024
@ljnelson ljnelson moved this from Triage to Sprint Scope in Backlog Oct 9, 2024
This was referenced Oct 10, 2024
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ljnelson ljnelson

Labels
4.x Version 4.x bug Something isn't working jpa/jta
Projects
Backlog
Status: Sprint Scope
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

4.x: Amends datasource-ucp to depend on ucp11 instead of ucp ljnelson/helidon
1 participant
@ljnelson