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 2020/11/17 18:45:03 UTC

[GitHub] [activemq-artemis] sebthom opened a new pull request #3347: ARTEMIS-3001 Provide address and queue count via ActiveMQServerControl

sebthom opened a new pull request #3347:
URL: https://github.com/apache/activemq-artemis/pull/3347


   See https://issues.apache.org/jira/browse/ARTEMIS-3001


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [activemq-artemis] asfgit closed pull request #3347: ARTEMIS-3001 Provide address and queue count via ActiveMQServerControl

Posted by GitBox <gi...@apache.org>.
asfgit closed pull request #3347:
URL: https://github.com/apache/activemq-artemis/pull/3347


   


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [activemq-artemis] clebertsuconic commented on pull request #3347: ARTEMIS-3001 Provide address and queue count via ActiveMQServerControl

Posted by GitBox <gi...@apache.org>.
clebertsuconic commented on pull request #3347:
URL: https://github.com/apache/activemq-artemis/pull/3347#issuecomment-729137448


   if it's not too much to ask, can you add 1 or 2 tests into ActiveMQServerControlTest, and also make sure ActiveMQServerControlUsingCoreTest would work (as you may have to do a quick tweak not he proxy)?


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [activemq-artemis] sebthom commented on pull request #3347: ARTEMIS-3001 Provide address and queue count via ActiveMQServerControl

Posted by GitBox <gi...@apache.org>.
sebthom commented on pull request #3347:
URL: https://github.com/apache/activemq-artemis/pull/3347#issuecomment-729153667


   @clebertsuconic I added 2 test cases as requested.


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [activemq-artemis] clebertsuconic commented on pull request #3347: ARTEMIS-3001 Provide address and queue count via ActiveMQServerControl

Posted by GitBox <gi...@apache.org>.
clebertsuconic commented on pull request #3347:
URL: https://github.com/apache/activemq-artemis/pull/3347#issuecomment-729154918


   @sebthom thank you.. I will merge this shortly.


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [activemq-artemis] clebertsuconic edited a comment on pull request #3347: ARTEMIS-3001 Provide address and queue count via ActiveMQServerControl

Posted by GitBox <gi...@apache.org>.
clebertsuconic edited a comment on pull request #3347:
URL: https://github.com/apache/activemq-artemis/pull/3347#issuecomment-729137448


   if it's not too much to ask, can you add 1 or 2 tests into ActiveMQServerControlTest, and also make sure ActiveMQServerControlUsingCoreTest would work (as you may have to do a quick tweak the proxy)?


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [activemq-artemis] clebertsuconic edited a comment on pull request #3347: ARTEMIS-3001 Provide address and queue count via ActiveMQServerControl

Posted by GitBox <gi...@apache.org>.
clebertsuconic edited a comment on pull request #3347:
URL: https://github.com/apache/activemq-artemis/pull/3347#issuecomment-729137448


   if it's not too much to ask, can you add 1 or 2 tests into ActiveMQServerControlTest, and also make sure ActiveMQServerControlUsingCoreTest would work (as you may have to do a quick tweak on the proxy)?


----------------------------------------------------------------
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:
users@infra.apache.org