You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@activemq.apache.org by gaohoward <gi...@git.apache.org> on 2018/05/07 07:16:10 UTC

[GitHub] activemq-artemis pull request #2074: ARTEMIS-1850 QueueControl.listDeliverin...

GitHub user gaohoward opened a pull request:

    https://github.com/apache/activemq-artemis/pull/2074

    ARTEMIS-1850 QueueControl.listDeliveringMessages returns empty result

    With AMQP protocol when some messages are received in a transaction,
    calling JMX QueueControl.listDeliveringMessages() returns empty list
    before the transaction is committed.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/gaohoward/activemq-artemis i_1850

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/activemq-artemis/pull/2074.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #2074
    
----
commit 4d804a1d7cb3bb25b6ab160a2a5cd7f636f1c4b3
Author: Howard Gao <ho...@...>
Date:   2018-05-07T06:33:16Z

    ARTEMIS-1850 QueueControl.listDeliveringMessages returns empty result
    
    With AMQP protocol when some messages are received in a transaction,
    calling JMX QueueControl.listDeliveringMessages() returns empty list
    before the transaction is committed.

----


---

[GitHub] activemq-artemis issue #2074: ARTEMIS-1850 QueueControl.listDeliveringMessag...

Posted by clebertsuconic <gi...@git.apache.org>.

Github user clebertsuconic commented on the issue:

    https://github.com/apache/activemq-artemis/pull/2074
  
    this seems to break this:
    
    Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.117 sec <<< FAILURE! - in org.apache.activemq.artemis.tests.integration.amqp.AmqpFailoverEndpointDiscoveryTest



---

[GitHub] activemq-artemis issue #2074: ARTEMIS-1850 QueueControl.listDeliveringMessag...

Posted by clebertsuconic <gi...@git.apache.org>.

Github user clebertsuconic commented on the issue:

    https://github.com/apache/activemq-artemis/pull/2074
  
    actually.. ignore my last comment.. i will run more tests.. probably unrelated


---

[GitHub] activemq-artemis issue #2074: ARTEMIS-1850 QueueControl.listDeliveringMessag...

Posted by gaohoward <gi...@git.apache.org>.

Github user gaohoward commented on the issue:

    https://github.com/apache/activemq-artemis/pull/2074
  
    @clebertsuconic Can you merge it? thx


---

[GitHub] activemq-artemis pull request #2074: ARTEMIS-1850 QueueControl.listDeliverin...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/activemq-artemis/pull/2074


---

[GitHub] activemq-artemis issue #2074: ARTEMIS-1850 QueueControl.listDeliveringMessag...

Posted by gaohoward <gi...@git.apache.org>.

Github user gaohoward commented on the issue:

    https://github.com/apache/activemq-artemis/pull/2074
  
    ok. btw I ran it within intellij it passes.


---

[GitHub] activemq-artemis issue #2074: ARTEMIS-1850 QueueControl.listDeliveringMessag...

Posted by mtaylor <gi...@git.apache.org>.

Github user mtaylor commented on the issue:

    https://github.com/apache/activemq-artemis/pull/2074
  
    @gaohoward @clebertsuconic Any update on this one?



---