gtully commented on pull request #3568: URL: https://github.com/apache/activemq-artemis/pull/3568#issuecomment-848724855
there is already a connection failure callback registered, the close(bool) was missing from the amqp callback, having that delegate to the amqp session context to give it a chance to close and cleanup seems to sort this. I think the session closeable can be safely removed now that close is in the mix in all cases. It is a bit of a web in there. have kicked off a full test run to verify. -- 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]
