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

classic Classic list List threaded Threaded
7 messages Options
Reply | Threaded
Open this post in threaded view
|

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

jbertram-2
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 <howard.gao@...>
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.

----


---
Reply | Threaded
Open this post in threaded view
|

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

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



---
Reply | Threaded
Open this post in threaded view
|

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

jbertram-2
In reply to this post by jbertram-2
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


---
Reply | Threaded
Open this post in threaded view
|

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

jbertram-2
In reply to this post by jbertram-2
Github user gaohoward commented on the issue:

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


---
Reply | Threaded
Open this post in threaded view
|

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

jbertram-2
In reply to this post by jbertram-2
Github user mtaylor commented on the issue:

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



---
Reply | Threaded
Open this post in threaded view
|

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

jbertram-2
In reply to this post by jbertram-2
Github user gaohoward commented on the issue:

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


---
Reply | Threaded
Open this post in threaded view
|

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

jbertram-2
In reply to this post by jbertram-2
Github user asfgit closed the pull request at:

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


---