Re: [xrblock] WGLC for draft-ietf-xrblock-rtcp-xr-burst-gap-discard-08.txt

[Colin Perkins <[e-mail]>]

On 6 Dec 2012, at 13:14, Romascanu, Dan (Dan) wrote:
> This is a (second) Working Group Last Call for http://www.ietf.org/id/draft-ietf-xrblock-rtcp-xr-burst-gap-discard-08.txt.  
> 
> Please read and review this document, and send your comments, questions and concerns to the WG list before December 20, 2012. If you read the document, have no comments and you believe that the document is ready for submission to the IESG as a Standards Track document please send a short message as well to help us in determining the level of review and consensus. 


This looks to be in good shape. Some minor points I noticed:

- The last paragraph of the definition of the Interval Metric Flag in Section 3.2 might be clearer written “Burst/Gap Discard Metrics can only be measured over definite intervals, and cannot be sampled. Accordingly, the value I=01, indicating a sampled value, MUST NOT be used.”

- For Packets discarded in bursts and Total packets expected in bursts, why are the values 0xFFFFFE and 0xFFFFFF listed as SHOULD? If there is a case where the condition is satisfied by the listed value isn’t reported, then this needs to be documented; otherwise the draft ought to say MUST instead.

- Section 3.3 reads as-if some new metrics are going to be defined, but all it actually does is reference another draft. I wonder if those summary metrics shouldn’t be defined in this draft, and if the split of related metrics into a separate draft is worthwhile in this case?

-- 
Colin Perkins
http://csperkins.org/