Last Call Review of draft-ietf-xrblock-rtcp-xr-post-repair-loss-count-07
review-ietf-xrblock-rtcp-xr-post-repair-loss-count-07-opsdir-lc-tsou-2014-12-28-00
Request | Review of | draft-ietf-xrblock-rtcp-xr-post-repair-loss-count |
---|---|---|
Requested revision | No specific revision (document currently at 11) | |
Type | Last Call Review | |
Team | Ops Directorate (opsdir) | |
Deadline | 2014-12-26 | |
Requested | 2014-12-15 | |
Authors | Rachel Huang , Varun Singh | |
I-D last updated | 2014-12-28 | |
Completed reviews |
Genart Last Call review of -07
by Tom Taylor
(diff)
Secdir Last Call review of -07 by Scott G. Kelly (diff) Opsdir Last Call review of -07 by Tina Tsou (Ting ZOU) (diff) |
|
Assignment | Reviewer | Tina Tsou (Ting ZOU) |
State | Completed | |
Request | Last Call review on draft-ietf-xrblock-rtcp-xr-post-repair-loss-count by Ops Directorate Assigned | |
Reviewed revision | 07 (document currently at 11) | |
Result | Has issues | |
Completed | 2014-12-28 |
review-ietf-xrblock-rtcp-xr-post-repair-loss-count-07-opsdir-lc-tsou-2014-12-28-00
Dear all, I have reviewed this document as part of the Operational directorate's ongoing effort to review all IETF documents being processed by the IESG. These comments were written primarily for the benefit of the operational area directors. Document editors and WG chairs should treat these comments just like any other last call comments. Intended status: standard track Summary: This document defines a new RTCP XR block to carry post repair loss count metrics. I found it is well written with no real concerns. I do have some minor comments and questions. - Section 1, 1st paragraph, last sentence, The overhead is Higher comparing with who? What is the overhead about? Suggested change is s/higher overhead/ high bandwidth consumption What do you mean by using multiple XR blocks, containing multiple XR blocks in the same RTCP XR compound packet or something else? - section 1, 2nd paragraph said: " In addition, another metric, repaired loss count, is also introduced in this report block for calculating the pre- repair loss count during the this range " Is the second metric pre-repair loss count? Isn't pre-repair loss count defined in RTCP Sender Reports (SR)/Receiver Reports (RR)[RFC3550].Also in the section 3,the second metric, i.e.,"loss repair count" is defined as "Total number of packets fully repaired after one or more loss-repair methods, e.g., FEC and/or retransmission during this interval" It is obvious the second metric is used to count the number of the packet loss after the repair method is used, i.e., post repair loss? - Section 3, definition of "unrepaired loss count" What interval are you referred to ?I think you use begin sequence number and end sequence number to explicitly indicate the actual sequence number range. So suggested change is s/during the interval/ during the sequence number interval. Happy Holidays! Thank you, Tina