registering common Camel types used in onException for refle… by JinyuChen97 · Pull Request #8505 · apache/camel-quarkus

JinyuChen97 · 2026-04-07T13:12:29Z

Found several common Camel exceptions that are frequently used in onException() handlers across multiple extensions. These are now registered centrally in camel-quarkus-core to avoid duplication and prevent ClassNotFoundException in native mode.

ExchangeTimedOutException
CamelAuthorizationException
CamelExecutionException
ThrottlerRejectedExecutionException

...tests/netty/src/main/java/org/apache/camel/quarkus/component/netty/tcp/NettyTcpResource.java

...oyment/src/main/java/org/apache/camel/quarkus/core/deployment/CamelNativeImageProcessor.java

aldettinger · 2026-04-08T13:30:49Z

There is a question whether integration-tests/netty is the right place for such a test ? Maybe something ala foundation/on-exception would be better ?

It would be good to have other advice here.

zhfeng · 2026-04-09T07:07:34Z

Sorry @JinyuChen97 , I merge the following PR and it could conflict with your changes.

Auto-detect onException/doCatch/throwException exception classes for native reflection #8508

jamesnetherton · 2026-04-09T07:12:36Z

Sorry @JinyuChen97 , I merge the following PR and it could conflict with your changes.

Auto-detect onException/doCatch/throwException exception classes for native reflection #8508

Yeah I think #8508 mostly supersedes this. What we could do instead in this PR, is search the codebase for usages of:

ExchangeTimedOutException
CamelAuthorizationException
CamelExecutionException
ThrottlerRejectedExecutionException

And remove any native reflection configuration and verify that the native tests continue to pass.

JinyuChen97 · 2026-04-09T09:04:00Z

Sorry @JinyuChen97 , I merge the following PR and it could conflict with your changes.

Auto-detect onException/doCatch/throwException exception classes for native reflection #8508

Yeah I think #8508 mostly supersedes this. What we could do instead in this PR, is search the codebase for usages of:

ExchangeTimedOutException CamelAuthorizationException CamelExecutionException ThrottlerRejectedExecutionException

And remove any native reflection configuration and verify that the native tests continue to pass.

Verified exception class registration using onException test. All exceptions are now auto-registered by the code change from #8508. The onException test handlers were removed after verification.
Only found one manually registered exception 'CamelAuthorizationException' in ShiroProcessor.java, which has been removed.

aldettinger · 2026-04-09T12:05:38Z

Don't we have more to cover like in MllpRoutes.java where we have @RegisterForReflection(targets = MllpInvalidMessageException.class, fields = false) ? If yes, could it make sense to modify the prompt to spot those cases ?

JinyuChen97 · 2026-04-09T13:32:07Z

Don't we have more to cover like in MllpRoutes.java where we have @RegisterForReflection(targets = MllpInvalidMessageException.class, fields = false) ? If yes, could it make sense to modify the prompt to spot those cases ?

oh yes, you are right, this exception is used by onException(class) pattern, so it can be removed as well.
I also found other exceptions annotated with @RegisterForReflection, but they are used by throwException(new xxxexception()) and they are not abled to be detected by the CamelNativeImageProcessor.

integration-tests/servlet/src/main/java/org/apache/camel/quarkus/component/servlet/CustomException.java
annotation: @RegisterForReflection
usage: throwException(new CustomException())

integration-test-groups/http/netty-http/src/main/java/org/apache/camel/quarkus/component/http/netty/NettyHttpRoutes.java
annotation: @RegisterForReflection(targets = IllegalStateException.class, serialization = true)
usage: throwException(new IllegalStateException("Forced exception"))

integration-tests-support/messaging/common/src/main/java/org/apache/camel/quarkus/component/messaging/it/MessagingCommonRoutes.java
annotation: @RegisterForReflection(targets = IllegalStateException.class, serialization = true)
usage: throw new IllegalStateException(...)

please suggest if you found more. And should we investigate the solution of adding throwException(new class) or leave them at the moment?

fixes apache#7608

aldettinger · 2026-04-09T14:38:42Z

Yes, there are probably more, a general review is probably needed. And maybe we could have in other DSLs, like xml dsl. Concerning throwException, it might be out of scope of this pr, maybe it could be separated to another ticket.

zhfeng · 2026-04-09T14:59:26Z

Well, I think `throwException` is supported as well. Alexandre Gallice ***@***.***> 于2026年4月9日周四 22:39写道：

…

*aldettinger* left a comment (apache/camel-quarkus#8505) <#8505 (comment)> Yes, there are probably more, a general review is probably needed. And maybe we could have in other DSLs, like xml dsl. Concerning throwException, it might be out of scope of this pr, maybe it could be separated to another ticket. — Reply to this email directly, view it on GitHub <#8505 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAJQHOYOK4F3T6H4V6YLBG34U6YYTAVCNFSM6AAAAACXPN7BQ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DEMJVGA4TONJVGU> . You are receiving this because you were mentioned.Message ID: ***@***.***>

JinyuChen97 · 2026-04-09T15:06:26Z

Well, I think throwException is supported as well. Alexandre Gallice @.> 于2026年4月9日周四 22:39写道：
…
aldettinger left a comment (apache/camel-quarkus#8505) <#8505 (comment)> Yes, there are probably more, a general review is probably needed. And maybe we could have in other DSLs, like xml dsl. Concerning throwException, it might be out of scope of this pr, maybe it could be separated to another ticket. — Reply to this email directly, view it on GitHub <#8505 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJQHOYOK4F3T6H4V6YLBG34U6YYTAVCNFSM6AAAAACXPN7BQ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DEMJVGA4TONJVGU . You are receiving this because you were mentioned.Message ID: @.>

throwException(Class) is supported, but the way like throwException(new instance) seems not supported , please correct me if i'm wrong.

zhfeng · 2026-04-09T15:12:10Z

Hmm, yeah, that could be the case. throwException(new instance) might not match the bytecode pattern. Feel free to invesigate it further if you like. JinyuChen97 ***@***.***> 于2026年4月9日周四 23:08写道：

…

*JinyuChen97* left a comment (apache/camel-quarkus#8505) <#8505 (comment)> Well, I think throwException is supported as well. Alexandre Gallice *@*. *> 于2026年4月9日周四 22:39写道： … <#m_987960130342204388_> aldettinger left a comment (apache/camel-quarkus#8505 <#8505>) <#8505 (comment) <#8505 (comment)>> Yes, there are probably more, a general review is probably needed. And maybe we could have in other DSLs, like xml dsl. Concerning throwException, it might be out of scope of this pr, maybe it could be separated to another ticket. — Reply to this email directly, view it on GitHub <#8505 (comment) <#8505 (comment)>>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJQHOYOK4F3T6H4V6YLBG34U6YYTAVCNFSM6AAAAACXPN7BQ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DEMJVGA4TONJVGU <https://github.com/notifications/unsubscribe-auth/AAJQHOYOK4F3T6H4V6YLBG34U6YYTAVCNFSM6AAAAACXPN7BQ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DEMJVGA4TONJVGU> . You are receiving this because you were mentioned.Message ID: @.*> throwException(Class) is supported, but the way like throwException(new instance) seems not supported , please correct me if i'm wrong. — Reply to this email directly, view it on GitHub <#8505 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAJQHOZ2GA4MFDKPYJ5D5634U64GBAVCNFSM6AAAAACXPN7BQ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DEMJVGI4DGNZSHE> . You are receiving this because you were mentioned.Message ID: ***@***.***>

aldettinger · 2026-04-09T16:34:29Z

Yes, that could be the scope of another ticket, created #8512. Thanks all for great collective work on this subject 👍

jamesnetherton · 2026-04-10T08:00:10Z

@aldettinger is this good to merge?

aldettinger · 2026-04-10T08:42:44Z

Not sure yet.

What about src/main/java/org/apache/camel/quarkus/eip/it/EipRoutes.java with @RegisterForReflection(targets = ThrottlerRejectedExecutionException.class) ?

jamesnetherton · 2026-04-10T08:53:29Z

Not sure yet.

What about src/main/java/org/apache/camel/quarkus/eip/it/EipRoutes.java with @RegisterForReflection(targets = ThrottlerRejectedExecutionException.class) ?

Seems that was taken care of in 76f2ba2.

aldettinger · 2026-04-10T09:49:03Z

And a last one is ./integration-tests/main-yaml/src/main/java/org/apache/camel/quarkus/main/CustomException.java.

This one is also used to test throwException by class but not not throwException by instance... So maybe it is a candidate for removal ?

JinyuChen97 · 2026-04-10T10:09:05Z

And a last one is ./integration-tests/main-yaml/src/main/java/org/apache/camel/quarkus/main/CustomException.java.

This one is also used to test throwException by class but not not throwException by instance... So maybe it is a candidate for removal ?

hmmm, I think I have removed it? (The main-yaml/CustomException in the change) it's the one used in yaml dsl.

aldettinger · 2026-04-10T11:11:07Z

@JinyuChen97 It's right, things were mixed up on my machine. Sorry, for the noise.

aldettinger reviewed Apr 7, 2026

View reviewed changes

...tests/netty/src/main/java/org/apache/camel/quarkus/component/netty/tcp/NettyTcpResource.java Outdated Show resolved Hide resolved

zhfeng reviewed Apr 7, 2026

View reviewed changes

...oyment/src/main/java/org/apache/camel/quarkus/core/deployment/CamelNativeImageProcessor.java Outdated Show resolved Hide resolved

JinyuChen97 force-pushed the camel-quarkus-7608 branch from d32473c to 4f59682 Compare April 7, 2026 15:14

jamesnetherton reviewed Apr 7, 2026

View reviewed changes

...oyment/src/main/java/org/apache/camel/quarkus/core/deployment/CamelNativeImageProcessor.java Outdated Show resolved Hide resolved

JinyuChen97 force-pushed the camel-quarkus-7608 branch from 4f59682 to 6eb4d09 Compare April 8, 2026 08:56

JinyuChen97 requested review from aldettinger and jamesnetherton April 8, 2026 08:57

JinyuChen97 force-pushed the camel-quarkus-7608 branch from 6eb4d09 to 8f674c2 Compare April 9, 2026 08:53

JinyuChen97 force-pushed the camel-quarkus-7608 branch from 8f674c2 to 774627e Compare April 9, 2026 13:32

remove exception reflective registration and verify with test

739e308

fixes apache#7608

JinyuChen97 force-pushed the camel-quarkus-7608 branch from 774627e to 739e308 Compare April 9, 2026 13:39

aldettinger mentioned this pull request Apr 9, 2026

Auto-detect throwException(instance) exception classes for native reflection #8512

Open

jamesnetherton approved these changes Apr 10, 2026

View reviewed changes

aldettinger approved these changes Apr 10, 2026

View reviewed changes

aldettinger merged commit cbb2634 into apache:main Apr 10, 2026
24 checks passed

Conversation

JinyuChen97 commented Apr 7, 2026 • edited by aldettinger Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

aldettinger commented Apr 8, 2026

Uh oh!

zhfeng commented Apr 9, 2026

Uh oh!

jamesnetherton commented Apr 9, 2026

Uh oh!

JinyuChen97 commented Apr 9, 2026

Uh oh!

aldettinger commented Apr 9, 2026

Uh oh!

JinyuChen97 commented Apr 9, 2026

Uh oh!

aldettinger commented Apr 9, 2026

Uh oh!

zhfeng commented Apr 9, 2026 via email

Uh oh!

JinyuChen97 commented Apr 9, 2026

Uh oh!

zhfeng commented Apr 9, 2026 via email

Uh oh!

aldettinger commented Apr 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

jamesnetherton commented Apr 10, 2026

Uh oh!

aldettinger commented Apr 10, 2026

Uh oh!

jamesnetherton commented Apr 10, 2026

Uh oh!

aldettinger commented Apr 10, 2026

Uh oh!

JinyuChen97 commented Apr 10, 2026

Uh oh!

aldettinger commented Apr 10, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

JinyuChen97 commented Apr 7, 2026 •

edited by aldettinger

Loading

aldettinger commented Apr 9, 2026 •

edited

Loading