You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@camel.apache.org by gsudharsan <gi...@git.apache.org> on 2018/06/19 12:49:23 UTC

[GitHub] camel pull request #2385: CAMEL-12588 - Improving the fix

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 <gs...@...>
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 <gs...@...>
Date:   2018-06-19T12:46:47Z

    CAMEL-12588 - Improving the fix 
    
    corrected to shutdown instead of shutdownNow

----


---

[GitHub] camel pull request #2385: CAMEL-12588 - Improving the fix

Posted by gsudharsan <gi...@git.apache.org>.

Github user gsudharsan closed the pull request at:

    https://github.com/apache/camel/pull/2385


---