[OASIS Issue Tracker] Commented: (AMQP-47) Messaging: Incorporate feedback from Steve Huston's review

[OASIS Issues Tracker <workgroup_mailer@lists.oasis-open.org>]

    [ http://tools.oasis-open.org/issues/browse/AMQP-47?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=29302#action_29302 ] 

David Ingham commented on AMQP-47:
----------------------------------

svn commit -m "Further fix for AMQP-47 - definition of distribution node" messaging.xml
Sending        messaging.xml
Transmitting file data .
Committed revision 95.

> Messaging: Incorporate feedback from Steve Huston's review
> ----------------------------------------------------------
>
>                 Key: AMQP-47
>                 URL: http://tools.oasis-open.org/issues/browse/AMQP-47
>             Project: OASIS Advanced Message Queuing Protocol (AMQP) TC
>          Issue Type: Improvement
>            Reporter: David Ingham
>            Assignee: David Ingham
>
> Feedback from Steve Huston's review of the Messaging chapter:
> From Steve Huston's review:
> Part 3, section 2, 3rd para: insert "three" before "sections" and insert a
> colon after "sections"
> Part 3, section 2, 4th para, 2nd sentence: insert comma after "That is"
> Part 3, section 2, 4th para, 3rd sentence: Change "may not" to "MUST NOT"?
> Part 3, section 2, 5th para bullets, "The body...": "either" should be
> "one of the following three choices" (either refers to two)
> Part 3, section 2.1, 1st para after XML, 2nd sentence: What are the
> default values to be assumed, or where are they described?
> Part 3, section 2.1, ttl: Quotes around "live" are both close quotes.
> First one should be an open quote. (`` in latex, I believe)
> Also, last sentence: insert comma after "i.e."
> Part 3, section 2.1, first-acquirer: Add reference to section 3.1?
> (acquiring has not been discussed before this point)
> Part 3, section 2.3, 1st para, 3rd sentence: Add comma after "e.g."
> Part 3, section 2.3, 2nd para, 1st sentence: change comma before "however"
> to a semi-colon; add comma after "however"
> Also, change "however it" to "however, in"
> Part 3, section 2.3, 2nd para, 2nd sentence: "some annotation" should be
> "some annotations" (add an s)
> Also, after "purpose" change hyphen to an em-dash and "may be being" to
> "could be"
> Part 3, section 2.4, 1st para after XML, 2nd sentence: change to "The
> properties section is part of the bare message; therefore, if
> retransmitted by an intermediary, it MUST remain unaltered."
> Part 3, section 2.4, content-encoding, throughout: "Content-Encoding"
> should not be capitalized, except when starting a sentence where it can be
> "Content-encoding".
> Also, 5th para, "Content-encoding MUST not be set..." should be
> "Content-encoding MUST NOT be set..."
> Part 3, section 2.5, 2nd para: insert comma after "that is" and remove
> parentheses.
> Part 3, section 3, 3.1: 
> Also, 1st para, 2nd sentence: change comma before "however" to a
> semi-colon and add a comma after "however"
> Part 3, section 3.1, 3rd para, 3rd sentence: add comma after "e.g."
> Also, "default-outcome" - this isn't defined - where does it come from?
> Part 3, section 4, 1st and 3rd para: "delivery-state" should be "delivery
> state"
> Part 3, section 4.4, 1st para, 2nd sentence: add comma after "i.e."
> Also, 4th sentence: change comma after "frames have been sent" to an
> em-dash
> Also, 5th sentence: "time bound" should be "time-bound"
> Part 3, section 4.5, 1st para, 4th sentence: change comma after "frames
> have been sent" to an em-dash
> Also, 5th sentence: "time bound" should be "time-bound" and add comma
> after "e.g."
> Part 3, section 4.6, 1st sentence: "In Part 2: 6.13 Resuming Deliveries"
> should be "Part 2 section 6.13 provides" and delete "was provided" at the
> end of the sentence.
> Part 3, section 4.6, 3rd para, 2nd sentence (same in 6th para, 2nd
> sentence and 8th para, 2nd sentence): Quotes around "at least once" and
> "at most once" are both close quotes; change leading one to an open quote.
> Also, change comma before "however" to semi-colon and add comma after.
> (3rd para and 8th para)
> Part 3, section 4.6, 8th para, 3rd sentence: should insert "have" between
> "not" and "enough"?
> Part 3, section 4.6, 14th para, 2nd sentence: change "since we know" to
> "since it is known" and "we have no need" to "there is no need".
> Part 3, section 4.6, 15th para, 1st sentence: change "we have to assume"
> to "it must be assumed".
> Part 3, section 4.6, 18th para: should "state Receivedsection-number=0" be
> "state Received, section number=0"?
> Part 3, section 5.1, 1st para, 2nd sentence: Refer to singular filter in
> the definition: "A filter can be thought of as a function which takes a
> message as input and returns a boolean value:"
> Part 3, section 5.1, 1st para, 2nd sentence: "filter" should not be
> italicized.
> Also, add comma after "e.g."
> Part 3, section 5.2, 3rd para, 1st sentence: add comma after "i.e."
> Part 3, section 5.2, 3rd para, 2nd sentence: "and thus anyway ineligible
> for transfer" - remove "anyway"
> Part 3, section 5.3, default-outcome, last sentence: Add comma after
> "e.g."
> Part 3, section 5.3, outcomes, last sentence: Add comma after "e.g." 
> Part 3, section 5.4, durable: "or not state at all." should be "or no
> state at all."
> Part 3, section 5.5, 2nd sentence: Remove " value" from the end.
> Part 3, section 5.8, 1st para, 2nd sentence: "ether" should be "either".
> Part 3, section 5.8, 1st para, 5th sentence: Add comma after "i.e."
> Part 3, section 5.9, 1st sentence: remove hyphen between "Node" and
> "used".
> Part 3, section 5.9, supported-dist-modes, 2nd para, 2nd sentence: Add
> comma after "That is". Remove "as" after "defined"

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://tools.oasis-open.org/issues/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira