Bug 796273

Summary: Priority ring queue lets lower-priority message displace higher-priority
Product: Red Hat Enterprise MRG Reporter: Alan Conway <aconway>
Component: qpid-cppAssignee: Gordon Sim <gsim>
Status: CLOSED ERRATA QA Contact: Matej Lesko <mlesko>
Severity: medium Docs Contact:
Priority: medium    
Version: 2.2CC: freznice, gsim, jross, zkraus
Target Milestone: 3.1Keywords: TestCaseProvided
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: qpid-cpp-0.22 Doc Type: Bug Fix
Doc Text:
A Ring Priority queue works by removing the lowest priority message if the queue limit is reached. It was discovered that the queue did not take into account whether the new message was a lower priority than the one being removed. A lower priority message could therefore displace a higher-priority message. A fix to the Priority Ring queue now correctly assesses the incoming message priority, and ensures that higher priority messages remain in the queue.
 Story Points: ---
Clone Of: Environment:
Last Closed: 2015-04-14 13:46:13 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
Test case none

Description Alan Conway 2012-02-22 15:29:53 UTC 
Description of problem:

Currently a ring+priority queue works by removing the lowest priority message if the limit is reached. However it does not consider whether the new message is actually of lower priority than the one being displaced. A lower priority message should not displace a higher-priority message.


Version-Release number of selected component (if applicable): 0-14


How reproducible: 100%


Steps to Reproduce: Run attached python test.
  
Actual results: fails - priority 2 message displaces priority 8 message

Expected results: passes
Comment 1 Alan Conway 2012-02-22 15:30:43 UTC 
Created attachment 565017 [details]
Test case
Comment 2 Gordon Sim 2013-04-25 11:41:52 UTC 
Fixed in qpid 0.20 as part of broker refactor.
Comment 5 Matej Lesko 2015-02-16 12:27:06 UTC 
Added test names.
Comment 7 errata-xmlrpc 2015-04-14 13:46:13 UTC 
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHEA-2015-0805.html