jooqx icon indicating copy to clipboard operation
jooqx copied to clipboard

Published 20 hours ago • verified publisher icon zero88

Support jooq 3.19.*

Open yueliangwen opened this issue 1 year ago • 6 comments

Is your feature request related to a problem? Please describe. Do you have any plan to support jooq 3.19.*? I have a try, but Caused by: java.lang.NoSuchMethodError: 'org.jooq.Converter org.jooq.Field.getConverter()'.

yueliangwen avatar Feb 24 '24 05:02 yueliangwen

Thank you for your report. Let me try my best to get back to you as soon as possible. I assumed you're using v2.0.0-rc1

zero88 avatar Feb 26 '24 06:02 zero88

Hi @yueliangwen I cannot reproduce your error. I thought there were some dependency conflicts, please make sure you override Vert.x and jOOQ version in your build descriptor (pom.xml or gradle.build)

I closed this ticket, however, if anything is unclear, please leave a message. Thanks

zero88 avatar Mar 02 '24 04:03 zero88

jooqx.tar.gz

Steps to Reproduce：

1、Download the jooqx.tar.gz 2、Configure the PostgreSQL instance image 3、Read the README.adoc, ./gradlew clean run 4、curl http://localhost:8888/dump, you will see the problem.

Change the jooq version to 3.17.21, there's no problem.

yueliangwen avatar Mar 02 '24 08:03 yueliangwen

thank you for the reproduction. That helps me a lot to detect a reason: since jooq >= 3.17 uses java 17 as the default. Not pretty sure why [email protected] is still fine, but jooq >= 3.18.x raises an error. Somehow, jooqx is not compatible anymore with jooq when running java 17, which leads to NoSuchMethodError at runtime.

I reproduced it by unit-test that already re-compiled jooqx on java17, then the error didn't happen

I will find a time this week to make a release to make jooqx lib compatible in java17+

zero88 avatar Mar 04 '24 06:03 zero88

Hi @yueliangwen Could you pls try jooqx version 2.0.0-rc.1:jdk17? Unfortunately, something's weird in gradle build, then I cannot keep the previous rc version.

  implementation("io.github.zero88:jooqx:2.0.0-rc.1:jdk17")
  implementation("io.github.zero88:jooqx-spi:2.0.0-rc.1:jdk17")

That is a matrix version

jooqx jooq jdk
2.0.0-rc1 < 3.18 any
2.0.0-rc.1:jdk17 >= 3.18 jdk17+

I will upgrade to jdk17 as default in the next release.

Thank you for your patience!!!

zero88 avatar Mar 16 '24 11:03 zero88

Hi @zero88 I rerun some simple examples, it works well. Thanks for your great job.

yueliangwen avatar Mar 16 '24 14:03 yueliangwen