You are viewing a plain text version of this content. The canonical link for it is here.
Posted to gitbox@activemq.apache.org by "u8675309 (via GitHub)" <gi...@apache.org> on 2023/06/12 18:13:48 UTC

[GitHub] [activemq] u8675309 opened a new pull request, #1031: Fix mem leak with STOMP unsubscribe

u8675309 opened a new pull request, #1031:
URL: https://github.com/apache/activemq/pull/1031

   (no comment)


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


[GitHub] [activemq] jbonofre commented on pull request #1031: Fix mem leak with STOMP unsubscribe

Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on PR #1031:
URL: https://github.com/apache/activemq/pull/1031#issuecomment-1593450654

   @u8675309 we have some flaky tests. To trigger a new build you can commit an empty commit and rebase/squash:
   
   ```
   git commit --allow-empty
   git rebase -i origin/main
   git push --force
   ```
   


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


[GitHub] [activemq] asfgit closed pull request #1031: Fix mem leak with STOMP unsubscribe

Posted by "asfgit (via GitHub)" <gi...@apache.org>.
asfgit closed pull request #1031: Fix mem leak with STOMP unsubscribe
URL: https://github.com/apache/activemq/pull/1031


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


[GitHub] [activemq] jbonofre commented on pull request #1031: Fix mem leak with STOMP unsubscribe

Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on PR #1031:
URL: https://github.com/apache/activemq/pull/1031#issuecomment-1594395095

   LGTM thanks. I gonna merge by hand to include a Jira ID.


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


[GitHub] [activemq] u8675309 commented on pull request #1031: Fix mem leak with STOMP unsubscribe

Posted by "u8675309 (via GitHub)" <gi...@apache.org>.
u8675309 commented on PR #1031:
URL: https://github.com/apache/activemq/pull/1031#issuecomment-1593446611

   It seems like the unit tests failed for an unrelated reason. Some KahaDB tests failed but the only change was related to STOMP. I'm not sure how to try the test again.


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