squakez commented on code in PR #21067:
URL: https://github.com/apache/camel/pull/21067#discussion_r2728294540
##########
components/camel-resilience4j/src/main/java/org/apache/camel/component/resilience4j/ResilienceProcessor.java:
##########
@@ -162,6 +162,12 @@ public PooledExchangeTask create(Exchange exchange,
AsyncCallback callback) {
@Override
protected void doStart() throws Exception {
+ if (executorService == null) {
+ // A default thread pool if none is provided.
+ executorService = camelContext.getExecutorServiceManager()
Review Comment:
After your comment I had a look and it seems that we are doing this very
same stuff when there is some timeout configured involved, see:
https://github.com/apache/camel/blob/87a82fa44e4db799233bd6eeb69a680ee458e378/components/camel-resilience4j/src/main/java/org/apache/camel/component/resilience4j/ResilienceReifier.java#L210-L226
I wonder if the test was wrong, and we should have instead specifying the
timeout capability?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
