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

Workaround DateTimeParseException in vertx-sql-client with JDK 22 #37683

Closed

Conversation

zakkak
Copy link
Contributor

@zakkak zakkak commented Dec 12, 2023

Workaround for #37208 till eclipse-vertx/vertx-sql-client#1379 gets fixed and released

Note that the substitution only happens if necessary (i.e. it won't
happen when using JDK < 22, and it won't happen once an upstream fix
becomes available.

Closes #37208

CI run testing this against Mandrel 24.0-dev based on JDK 22-ea: https://github.com/graalvm/mandrel/actions/runs/7178673001

Workaround for quarkusio#37208 till
eclipse-vertx/vertx-sql-client#1379 gets fixed
and released

Note that the substitution only happens if necessary (i.e. it won't
happen when using JDK < 22, and it won't happen once an upstream fix
becomes available.

Closes quarkusio#37208
@zakkak
Copy link
Contributor Author

zakkak commented Dec 12, 2023

@jerboaa could you please review this?

@zakkak
Copy link
Contributor Author

zakkak commented Dec 12, 2023

Unfortunately it doesn't work without loading the class at some point (I mistakenly tested it locally with Mandrel 23.1.1 instead of 24.0-dev)

@zakkak zakkak closed this Dec 12, 2023
@quarkus-bot quarkus-bot bot added the triage/invalid This doesn't seem right label Dec 12, 2023
@zakkak zakkak deleted the 2023-12-12-work-around-vertx-sql-issue branch December 12, 2023 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/vertx triage/invalid This doesn't seem right
Projects
None yet
1 participant