-
Notifications
You must be signed in to change notification settings - Fork 160
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
[restful-ws] Doesn't work with Quarkus Reactive in Native mode #546
Comments
Interestingly, when running in JVM mode, which works, the stacktrace is different (I dumped it manually):
In the Native mode, the stacktrace has the following:
|
hmm, I'm not sure, we're not doing anything special to be honest |
For sure, the failure is caused by the fact the code tries to use reflection (here: https://github.com/quarkusio/quarkus/blob/2.13.5.Final/independent-projects/resteasy-reactive/server/runtime/src/main/java/org/jboss/resteasy/reactive/server/spi/ResteasyReactiveResourceInfo.java#L67) In Native mode (Graalvm compiled binary) reflection doesn't work. That's why we get "method not found" exception. The code should use the code index (see: https://github.com/smallrye/jandex), built at compile time by Quarkus deployment plugins, for all lookups. However, I don't know what CloudEvents SDK is doing here wrong... |
I've posted a question about this on the Quarkus chat: https://quarkusio.zulipchat.com/#narrow/stream/306418-resteasy-reactive/topic/CloudEvents.20lib.20doesn't.20work.20in.20native.20mode/near/349160477 |
@cardil is there anything else we should do here? |
quarkusio/quarkus#33267 should have resolved this issue. I will confirm... |
Works on Quarkus |
The library works well with Quarkus RESTEasy Reactive. Also, the Reactive REST Client works. But, in Native mode the app throws:
You can see in the logs the app has started, and the request was properly sent. The exception happens on server-side.
/kind bug
The text was updated successfully, but these errors were encountered: