gtully commented on pull request #3568: URL: https://github.com/apache/activemq-artemis/pull/3568#issuecomment-839815064
the closable go away when the connection closes, the issue/leak appears when there are repeated producer open/close on the same connection. It may need more tests that verify error conditions, I don't think those exist at the moment, there was one covering abort. I will write some more tests and follow up with the commit. At first look it seemed the closeable was just a catch all, and it hid the fact that the close was not always called. I think close should be called, b/c we already track the producer senders and we need to remove those in any event. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
