-
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
Hot reload fails when using BeanParam #45625
Comments
/cc @FroMage (rest), @stuartwdouglas (rest) |
@FroMage sounds like one for you |
I am experiencing this same issue with a record class annotated with @BeanParam in the @post REST method. Initial startup with "quarkus dev" allows the REST call to succeed when called multiple times, but a code change triggering dynamic reload of running system results in an invocation target exception. Each field specified in the record parameters is annotated with @restform. Some relevant parts of stack trace: java.lang.RuntimeException: Failed to find target generated factory method on record @BeanParam type |
I'll have a look |
Fix @BeanParam handling during hot reload
Fixes: quarkusio#45625 (cherry picked from commit 3a60175)
Describe the bug
Every usage of BeanParam causes hot reload to fail and requires manual reload of the app.
I'm running graalvm 23
Expected behavior
When using BeanParam for the first time, everything works as expected including after a small change in the code when the hot reload kicks in.
Actual behavior
When I change anything in the code while the app is running, the hot reload fails on the next request to the service with the following (shortened) exception:
To make it work again, it requires a manual restart.
How to Reproduce?
https://github.com/FreifeldRoyi/bean-param-issue
Output of
uname -a
orver
No response
Output of
java -version
No response
Quarkus version or git rev
3.17.6
Build tool (ie. output of
mvnw --version
orgradlew --version
)No response
Additional information
No response
The text was updated successfully, but these errors were encountered: