You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@activemq.apache.org by michaelandrepearce <gi...@git.apache.org> on 2018/08/06 16:42:21 UTC
[GitHub] activemq-artemis pull request #2217: FOR jbertram - ARTEMIS-1545 Memory leak
GitHub user michaelandrepearce opened a pull request:
https://github.com/apache/activemq-artemis/pull/2217
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:
https://github.com/apache/activemq-artemis/pull/2217.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 #2217
----
commit 74f6f1887e9c3234812d2b0997c97d30164b985e
Author: Michael André Pearce <mi...@...>
Date: 2017-12-14T07:47:30Z
ARTEMIS-1545 Support JMS 2.0 Completion Listener for Exceptions
commit a2da41ee2e347bcf8fe721bded89e4a78ca14cfb
Author: Justin Bertram <jb...@...>
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 <mi...@...>
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.
----
---
[GitHub] activemq-artemis pull request #2217: FOR jbertram - ARTEMIS-1545 Memory leak
Posted by michaelandrepearce <gi...@git.apache.org>.
Github user michaelandrepearce closed the pull request at:
https://github.com/apache/activemq-artemis/pull/2217
---