You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by "Alex Rudyy (JIRA)" <ji...@apache.org> on 2017/04/27 16:18:04 UTC

[jira] [Resolved] (QPID-7699) ConsumerPriorityTest#testLowPriorityConsumerDoesNotReceiveMessagesIfHigherPriorityConsumerAvailable occasionally fails on 0-10 path

     [ https://issues.apache.org/jira/browse/QPID-7699?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Alex Rudyy resolved QPID-7699.
------------------------------
    Resolution: Duplicate

I looked through the broker logs and it seems that the test failure was caused by the client bug already fixed on trunk as part of QPID-7505. 
The below is my analysis of the issue.
The test tries to recieve from empty queue to verify that there is no messages there. As part of synchronous received a command MessageFlush was issued which removed the credits on broker side. The JMS client sent  MessageFlow  to restore credits immediately after MessageFlush  but Broker handled it after receiving message. As result, at the moment of message receiving the consumer with highest priority did not have credits and message was sent to the consumer with lower priority. 
In order to reduce the noise from failing tests on 6.x branches I suggest to port the changes made in QPID-7505 into both 6.0.x and 6.1.x branches

> ConsumerPriorityTest#testLowPriorityConsumerDoesNotReceiveMessagesIfHigherPriorityConsumerAvailable occasionally fails on 0-10 path
> -----------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: QPID-7699
>                 URL: https://issues.apache.org/jira/browse/QPID-7699
>             Project: Qpid
>          Issue Type: Bug
>          Components: Java Broker, Java Client
>    Affects Versions: qpid-java-6.1.1
>            Reporter: Keith Wall
>             Fix For: qpid-java-broker-7.0.0
>
>         Attachments: TEST-org.apache.qpid.server.queue.ConsumerPriorityTest.testLowPriorityConsumerDoesNotReceiveMessagesIfHigherPriorityConsumerAvailable.txt
>
>
> We occasionally see the following test failure on CI.  The message gets delivered to the wrong consumer.  The root cause has not been investigated.
> https://builds.apache.org/view/M-R/view/Qpid/job/Qpid-Java-Java-BDB-6.1.x-TestMatrix/lastCompletedBuild/Ubuntu%20Slaves=ubuntu,jdk=JDK%201.7%20(latest),profile=java-bdb.0-10/testReport/org.apache.qpid.server.queue/ConsumerPriorityTest/testLowPriorityConsumerDoesNotReceiveMessagesIfHigherPriorityConsumerAvailable/
> {noformat}
> Message should not go to the low priority consumer
> Stacktrace
> junit.framework.AssertionFailedError: Message should not go to the low priority consumer
> 	at junit.framework.Assert.fail(Assert.java:57)
> 	at junit.framework.Assert.assertTrue(Assert.java:22)
> 	at junit.framework.Assert.assertNull(Assert.java:277)
> 	at junit.framework.TestCase.assertNull(TestCase.java:447)
> 	at org.apache.qpid.server.queue.ConsumerPriorityTest.doTestLowPriorityConsumerDoesNotReceiveMessagesIfHigherPriorityAvailable(ConsumerPriorityTest.java:102)
> 	at org.apache.qpid.server.queue.ConsumerPriorityTest.testLowPriorityConsumerDoesNotReceiveMessagesIfHigherPriorityConsumerAvailable(ConsumerPriorityTest.java:77)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org