Bug 800322

Summary: qpid.msg_sequence wrong description in user guide
Product: Red Hat Enterprise MRG Reporter: Pavel Moravec <pmoravec>
Component: Messaging_Programming_ReferenceAssignee: Joshua Wulf <jwulf>
Status: CLOSED CURRENTRELEASE QA Contact: ecs-bugs
Severity: low Docs Contact:
Priority: low    
Version: 2.1CC: chetan, dryan, lcarlon, rchidamb, rdassen
Target Milestone: 2.2   
Target Release: ---   
Hardware: All   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-09-20 03:13:10 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 785156    

Description Pavel Moravec 2012-03-06 10:13:17 UTC 
Description of problem:
qpid.msg_sequence in "Exchange Declaration Parameters" section (Section 2.2) has wrong / confusing description. In particular this sentence is wrong:
"The sequence number for the first message sent to the exchange is 0, .."

Three changes are relevant:
1) Replace 0 by 1. As the very first msq_sequence in a message is really 1 (to check this, see Exchange::PreRoute::PreRoute method where parent->sequenceNo++; is called before setting msg_sequence to a message).
2) Replace "to the exchange" by "from / by the exchange". As msg_sequence is inserted to messages "leaving" the exchange, not arriving the exchange.
3) ??? "message sent" -> "message routed" as technically the exchange does not _send_ a message, it "just" routes them to queues. But changing this can cause some confusion to a reader so I am not sure if it is linguistically ok..

Also it makes sense to add there a notice that qpid.msg_sequence is reset to 1 when qpid broker is restarted.


Version-Release number of selected component (if applicable):
MRG 2.0


How reproducible:
n.a.


Steps to Reproduce:
n.a.

  
Actual results:
n.a.


Expected results:
n.a.


Additional info:
Comment 1 Raghu Udiyar 2012-03-06 10:31:28 UTC 
Upstream docs need an update as well :

https://cwiki.apache.org/qpid/cheat-sheet-for-configuring-exchange-options.html
https://cwiki.apache.org/confluence/display/qpid/Qpid+extensions+to+AMQP
Comment 4 Cheryn Tan 2012-09-20 02:53:31 UTC 
Released for MRG 2.2