Uploaded image for project: 'OASIS Advanced Message Queuing Protocol (AMQP) TC'
  1. OASIS Advanced Message Queuing Protocol (AMQP) TC
  2. AMQP-47

Messaging: Incorporate feedback from Steve Huston's review

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: WD03
    • Component/s: None
    • Labels:
      None

      Description

      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"

        Attachments

          Activity

            People

            • Assignee:
              dingham David Ingham (Inactive)
              Reporter:
              dingham David Ingham (Inactive)
            • Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: