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

Update dependencies #89

Closed
jdussouillez opened this issue Aug 19, 2022 · 4 comments
Closed

Update dependencies #89

jdussouillez opened this issue Aug 19, 2022 · 4 comments

Comments

@jdussouillez
Copy link

There are some updates of dependencies pending in the MRs (#81, #87, #88) that would be nice to have because they're a bit old.

  • This plugin still uses Quarkus v2.5.1 (released in December 2021)
  • Also it still uses jOOQ v3.15.5 (released in December 2021). The v3.17.3 contains a lot of small fixes that could be very useful for us.

Thanks

@jdussouillez
Copy link
Author

Any updates ?

@jdussouillez
Copy link
Author

jdussouillez commented Dec 7, 2022

I just noticed that the 3.17+ jOOQ versions are not compatible with JDK11 (in Open source edition, it is with the pro versions).

In Open source edition, it must be migrated to 3.16+ only. See https://www.jooq.org/download/versions#oss

For now I guess it's only possible to upgrade the Maven version in JDK 11 (8 currently in pom.xml, pipeline build is run in 11) and update to jOOQ 3.16.11

@escay
Copy link

escay commented Dec 28, 2022

A dependency problem when using jOOQ with Quarkus 3.0.0.Alpha2 I run into is the javax to jakarta package change.
My jOOQ code compiles, but running the Quarkus code fails on runtime with:

Caused by: io.quarkus.builder.BuildException: Build failure: Build failed due to errors
	[error]: Build step io.quarkiverse.jooq.deployment.JooqProcessor#build threw an exception: java.lang.NoClassDefFoundError: javax/enterprise/context/ApplicationScoped
	at io.quarkiverse.jooq.deployment.JooqProcessor.createDslContextProducerBean(JooqProcessor.java:117)
	at io.quarkiverse.jooq.deployment.JooqProcessor.build(JooqProcessor.java:83)
	at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:104)
	at java.base/java.lang.reflect.Method.invoke(Method.java:578)
	at io.quarkus.deployment.ExtensionLoader$3.execute(ExtensionLoader.java:909)
	at io.quarkus.builder.BuildContext.run(BuildContext.java:281)
	at org.jboss.threads.ContextHandler$1.runWith(ContextHandler.java:18)
	at org.jboss.threads.EnhancedQueueExecutor$Task.run(EnhancedQueueExecutor.java:2449)
	at org.jboss.threads.EnhancedQueueExecutor$ThreadBody.run(EnhancedQueueExecutor.java:1478)
	at java.base/java.lang.Thread.run(Thread.java:1589)
	at org.jboss.threads.JBossThread.run(JBossThread.java:501)

This might relate to issue #30 where it is stated that the latest jOOQ version no longer depends on javax packages.
(In the mean time Quarkus 3 is moving to JPA 3.1: quarkusio/quarkus#26986)

So perhaps updating the dependency to jOOQ v3.16 or newer and changing this plugin code to no longer use javax in JooqProcessor might solve this item as well.

escay added a commit to escay/demo-applications that referenced this issue Dec 28, 2022
Trying quarkus-jooq plugin failed. Exception logged in quarkiverse/quarkus-jooq#89. This approach is using a connection from AgroalDataSource instead of injecting DSLContext
@jdussouillez
Copy link
Author

Released in v0.4.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants