Skip to content

Conversation

@llowinge
Copy link
Contributor

No description provided.

@llowinge llowinge force-pushed the stabilize-throttle-test branch 2 times, most recently from 836ce96 to 68b6636 Compare July 30, 2025 14:30
@jamesnetherton
Copy link
Contributor

Native mode failure:

java.lang.ClassNotFoundException: org.apache.camel.processor.ThrottlerRejectedExecutionException

ThrottlerRejectedExecutionException needs to be registered for reflection. I forget if we typically just do this in the test or configure it in our extensions so that it works automatically OOTB. I'll check...

@jamesnetherton
Copy link
Contributor

I think we can add ThrottlerRejectedExecutionException here:

private static final List<Class<?>> CAMEL_REFLECTIVE_CLASSES = Arrays.asList(

@llowinge llowinge force-pushed the stabilize-throttle-test branch 2 times, most recently from 4f3c0c6 to 519af39 Compare August 1, 2025 07:10
@llowinge
Copy link
Contributor Author

llowinge commented Aug 1, 2025

After discussion with @jamesnetherton we will not add that exception as default. Only if user want to use that exception, it should be manually registered for native, given class registration is not for free and without usage it doesn't make sense.

@llowinge llowinge force-pushed the stabilize-throttle-test branch from 519af39 to 07be2d1 Compare August 1, 2025 08:43
@jamesnetherton jamesnetherton merged commit 283e056 into apache:main Aug 1, 2025
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants