[GitHub] activemq-artemis pull request #1867: ARTEMIS-1679 Stopping consumer no need ...

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] activemq-artemis pull request #1867: ARTEMIS-1679 Stopping consumer no need ...

jbertram-2
GitHub user wy96f opened a pull request:

    https://github.com/apache/activemq-artemis/pull/1867

    ARTEMIS-1679 Stopping consumer no need to wait until all queued tasks…

    Sping's SimpleMessageListenerContainer shutdown will stop consumers and then close them. It will take lots of time(each consumer costs 10s at most) if there are lots of consumers bcs stop process will wait until all queued tasks done. We assume that waiting for the current task is enough and this will accelerate spring shutting down process.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/wy96f/activemq-artemis consumer-stop

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/activemq-artemis/pull/1867.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 #1867
   
----
commit 3b882d05c31ec8401cfcaa61ec4ad69f04c9f562
Author: yang wei <wy96fyw@...>
Date:   2018-02-13T04:44:10Z

    ARTEMIS-1679 Stopping consumer no need to wait until all queued tasks done

----


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] activemq-artemis issue #1867: ARTEMIS-1679 Stopping consumer no need to wait...

jbertram-2
Github user clebertsuconic commented on the issue:

    https://github.com/apache/activemq-artemis/pull/1867
 
    I will merge this.. but if there's any failures from the full testsuite.. I may have to revert this.


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] activemq-artemis pull request #1867: ARTEMIS-1679 Stopping consumer no need ...

jbertram-2
In reply to this post by jbertram-2
Github user asfgit closed the pull request at:

    https://github.com/apache/activemq-artemis/pull/1867


---