[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[GitHub] activemq-artemis pull request #2217: FOR jbertram - ARTEMIS-1545 Memory leak

GitHub user michaelandrepearce opened a pull request:


    FOR jbertram - ARTEMIS-1545  Memory leak

    @jbertram here is what i was trying to explain as a bit of code about the leaking objects in the response cache when relying on the confirmation handler logic, for the missing responses. If we need to do this, then we must handle the response via the response cache.

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

    $ git pull https://github.com/michaelandrepearce/activemq-artemis Memory

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


To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #2217
commit 74f6f1887e9c3234812d2b0997c97d30164b985e
Author: Michael André Pearce <michael.andre.pearce@...>
Date:   2017-12-14T07:47:30Z

    ARTEMIS-1545 Support JMS 2.0 Completion Listener for Exceptions

commit a2da41ee2e347bcf8fe721bded89e4a78ca14cfb
Author: Justin Bertram <jbertram@...>
Date:   2018-07-17T15:53:21Z

    ARTEMIS-1545 refactor & rework a few incompatible pieces
    Existing commit for ARTEMIS-1545 broke bridges and large messages. This
    commit fixes those, and refactors the solution a bit to be more clear.

commit ffed219e6cd60e9c13f462750c037554c495024f
Author: Michael André Pearce <michael.andre.pearce@...>
Date:   2018-08-06T16:38:05Z

    Test case to demo, memory/object leak
    Demonstrate the response cache is not removed when relying on confirm handler, thus holding onto response handlers leaking memory/objects over time.