[ 
https://issues.apache.org/jira/browse/QPID-572?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12621765#action_12621765
 ] 

Rob Godfrey commented on QPID-572:
----------------------------------

As described in QPID-950, the subscriber now uses a pointer to the strictly 
ordered queue and will only attempt delivery from the queue - it cannot be 
"short-circuited".  On message arrival a fast-path is used if it is detected 
that the message would be the next one that a subscription would consider 
delivering.  See the description of QPID-950 for an overview of the design of 
the new queuing model.

> broker delivers messages out of order
> -------------------------------------
>
>                 Key: QPID-572
>                 URL: https://issues.apache.org/jira/browse/QPID-572
>             Project: Qpid
>          Issue Type: Bug
>          Components: Java Broker
>    Affects Versions: M2, M2.1, M3
>            Reporter: Rafael H. Schloming
>            Assignee: Rob Godfrey
>             Fix For: M3
>
>         Attachments: QPID-572-testcase.patch
>
>
> ConcurrentSelectorDeliveryManager will sometimes deliver messages out of 
> order. This is caused by the code in deliver(...) that attempts to 
> short-circuit message queuing when there is an available subscription. This 
> code can result in the currently published message skipping ahead of queued 
> messages causing out of order delivery. Although unrelated to transactions, I 
> have observed this failure occuring in TransactedTest both in testCommit and 
> testRollback. Normally it does not happen very frequently, however placing a 
> Thread.sleep(500) in the async delivery thread will cause the failure to 
> occur almost all the time.
> I tried fixing the problem by only attempting synchronous delivery when there 
> are no queued messages, however this appears to break other tests that use 
> selectors. This makes me suspect that the selector implementation is somehow 
> incorrectly coupled to synchronous delivery.
> I have only verfied this issue on the trunk, however I believe it effects M2 
> as well.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to