-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Bump DB2 Driver to 12.1.0.0 #44752
Bump DB2 Driver to 12.1.0.0 #44752
Conversation
You were able to compile to native? The Dependabot update was complaining? |
I was waiting for CI to do that because there is no support for DB2 DevService in
|
You will probably end up with the error in #44665 (see my comment). |
🎊 PR Preview e77760f has been successfully built and deployed to https://quarkus-pr-main-44752-preview.surge.sh/version/main/guides/
|
@gsmet it passes now |
34ecb90
to
bcadf36
Compare
...jdbc-db2/runtime/src/main/java/io/quarkus/jdbc/db2/runtime/graal/ConnectionSubstitution.java
Outdated
Show resolved
Hide resolved
It also uses the jakarta.transaction packages, so no need to transform it
This comment has been minimized.
This comment has been minimized.
@gsmet I found a nicer solution |
Status for workflow
|
Now it fails with: Caused by: java.lang.UnsupportedOperationException: 'TCP_KEEPIDLE' not supported
at java.base@21.0.5/sun.nio.ch.NioSocketImpl.setOption(NioSocketImpl.java:959)
at java.base@21.0.5/java.net.DelegatingSocketImpl.setOption(DelegatingSocketImpl.java:144)
at java.base@21.0.5/java.net.Socket.setOption(Socket.java:2009)
at com.ibm.db2.jcc.t4.w.a(w.java:76)
at com.ibm.db2.jcc.t4.az.a(az.java:441)
... 21 more It's impossible to fix it without the sources, so I'll leave it to someone who knows the JDBC driver better to perform this update. Closing |
Just out of curiousity, what did you do to get the |
@Sanne I bumped the devservice image to 12.1.0.0 too, do you think it may be related? |
I don't know about the devservice.. I couldn't try this out, just curious how you got the |
Seems to happen only in native. Logs: https://github.com/quarkusio/quarkus/actions/runs/12037826792/job/33562670423?pr=44752 |
I wonder if they somehow get this value by reflection cause it's new in Java 21+ and thus we would need to add it. |
ah if it's just native then I might have a hunch related to class initialization. I'll try to look into it but no promises |
I've found this GralVM issue which looks related and has a possible solution, but it didn't work for me in this case : Weird, I've asked the Mandrel team for suggestions. |
It also uses the
jakarta.transaction
packages, so no need to transform it