GitHub user gsudharsan opened a pull request:
https://github.com/apache/camel/pull/2385
CAMEL-12588 - Improving the fix
doStop() method now shuts down the timeoutCheckerExecutorService pool too
Improvment - used shutdown instead of shutdownNow
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/gsudharsan/camel master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/camel/pull/2385.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2385
----
commit b26f9fca9e48a1afdbbfc6ad576455e2e3c6c0c3
Author: Sudharsan Govindarajan <gsudharsan@...>
Date: 2018-06-19T11:37:52Z
Cleanup TimeOutChecker threads
When a Aggregator route is stopped, only the AggregateRecoveryChecker
threads are stopped and not the AggregateTimeoutChecker threads. They keep
lingering around .
commit 8c1b0ddfc0c913f109de746d007ff89e2ae1d2b3
Author: Sudharsan Govindarajan <gsudharsan@...>
Date: 2018-06-19T12:46:47Z
CAMEL-12588 - Improving the fix
corrected to shutdown instead of shutdownNow
----
---
