You are viewing a plain text version of this content. The canonical link for it is here.
Posted to gitbox@activemq.apache.org by GitBox <gi...@apache.org> on 2022/02/23 12:08:37 UTC

[GitHub] [activemq] cshannon edited a comment on pull request #778: Fix TempUsage memory leak

cshannon edited a comment on pull request #778:
URL: https://github.com/apache/activemq/pull/778#issuecomment-1048705020


   Thanks for the PR, this looks like a good catch. A couple things: Pull requests should have a Jira associated with it describing the issue. https://issues.apache.org/jira/projects/AMQ . Also, ideally it would be good to include a unit test that can demonstrate the issue and show that it's fixed.
   
   In this case I'll create the Jira as I am going to re-work the fix slightly by adding a test real quick because we are about to do a new release but for the future you can keep that in mind.
   
   Seems like this issue is due to: [AMQ-7085](https://issues.apache.org/jira/browse/AMQ-7085)


-- 
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.

To unsubscribe, e-mail: gitbox-unsubscribe@activemq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org