-
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
[GraalVM 22.3] smallrye-graphql native integration tests fails from quarkus main branch #36975
Comments
/cc @Karm (mandrel), @Ladicek (smallrye), @galderz (mandrel), @jmartisk (graphql,smallrye), @phillip-kruger (graphql,smallrye), @radcortez (smallrye), @zakkak (mandrel) |
This seems to be related to this line: https://github.com/smallrye/smallrye-graphql/blob/3b1c333e3af25b471831779ad6853186bccf7c33/server/implementation/src/main/java/io/smallrye/graphql/transformation/CalendarTransformer.java#L32 but how this leads to |
Given it seems related to a GraalVM change, it might be an issue in GraalVM. Maybe let's wait for feedback from the Mandrel team before digging on this one. |
Not sure why this shows up now and not before. The reason we see it with 22.3 Mandrel and not with newer releases is oracle/graal#6089 (which is in 23.0 and newer). |
It might not be quarkus related. A simple reproducer is:
I'm looking into it. |
Should be fixed once graalvm/mandrel#616 merges. |
Fixed with graalvm/mandrel#616 |
Describe the bug
In our Mandrel CI we see some new (?) failures related to graphql with a
22.3.5-dev
build using OpenJDK17.0.10+1
EA on Quarkus main branch.The failure looks like this:
See: https://github.com/graalvm/mandrel/actions/runs/6810325480/job/18519907861?pr=610#step:12:1295
Steps to reproduce is to run the native smallrye graphql IT tests on a recent-ish 22.3 mandrel build (or use this dev build).
The text was updated successfully, but these errors were encountered: