Last Call Review of draft-ietf-rmcat-coupled-cc-06
review-ietf-rmcat-coupled-cc-06-opsdir-lc-wang-2017-08-22-00
Request | Review of | draft-ietf-rmcat-coupled-cc |
---|---|---|
Requested revision | No specific revision (document currently at 09) | |
Type | Last Call Review | |
Team | Ops Directorate (opsdir) | |
Deadline | 2017-08-28 | |
Requested | 2017-08-14 | |
Authors | Safiqul Islam , Michael Welzl , Stein Gjessing | |
I-D last updated | 2017-08-22 | |
Completed reviews |
Opsdir Last Call review of -06
by Zitao Wang
(diff)
Secdir Last Call review of -06 by Chris M. Lonvick (diff) Genart Last Call review of -06 by Matthew A. Miller (diff) |
|
Assignment | Reviewer | Zitao Wang |
State | Completed | |
Request | Last Call review on draft-ietf-rmcat-coupled-cc by Ops Directorate Assigned | |
Reviewed revision | 06 (document currently at 09) | |
Result | Has nits | |
Completed | 2017-08-22 |
review-ietf-rmcat-coupled-cc-06-opsdir-lc-wang-2017-08-22-00
Reviewer: Zitao Wang (Michael) Review result: Has Nits 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 with the intent of improving the operational aspects of the IETF drafts. Comments that are not addressed in last call may be included in AD reviews during the IESG review. Document editors and WG chairs should treat these comments just like any other last call comments. Document reviewed: draft-ietf-rmcat-coupled-cc-06 Summary: When multiple congestion controlled RTP sessions traverse the same network bottleneck, combining their controls can improve the total on-the-wire behavior in terms of delay, loss and fairness. This document describes such a method for flows that have the same sender, in a way that is as flexible and simple as possible while minimizing the amount of changes needed to existing RTP applications. It specifies how to apply the method for the NADA congestion control algorithm, and provides suggestions on how to apply it to other congestion control algorithms. I think the document make sense and is written very clear, except some small nits: Page 1: The first sentence: “When multiple congestion controlled RTP sessions...”. I suggest expanding the “RTP”, like Real-time Transport Protocol (RTP), since it first appear in the document. Page 1: The last sentence: “It specifies how to apply the method for the NADA congestion control...”. I suggest expanding the “NADA”, like Network-Assisted Dynamic Adaptation (NADA). The reason same to above item. Page 3: The first paragraph: “sometimes the rate is increased further, until packets are ECN-marked or dropped.” I suggest adding a reference to help the readers understanding “ECN-marked”. Page 3: Suggest adding a term definition: “Flow State Identifiers (FSIs)” which be used in section 4 but not be introduced in the section 2 Definitions. Page 11: 6.1 NADA -- " Network-Assisted Dynamic Adapation (NADA) [I-D.ietf-rmcat-nada] is a congestion control scheme for rtcweb." I suggest adding a reference or some sentence to help the readers understand the “rtcweb”. A run of idnits revealed there were 0 error, 3 warning and 2 comments: Miscellaneous warnings: ---------------------------------------------------------------------------- -- The document date (March 28, 2017) is 140 days in the past. Is this intentional? -- Found something which looks like a code comment -- if you have code sections in the document, please surround them with '<CODE BEGINS>' and '<CODE ENDS>' lines. Checking references for intended status: Experimental ---------------------------------------------------------------------------- == Outdated reference: A later version (-04) exists of draft-ietf-rmcat-nada-03 == Outdated reference: A later version (-05) exists of draft-ietf-rmcat-eval-test-04 == Outdated reference: A later version (-08) exists of draft-ietf-rmcat-sbd-04 Summary: 0 errors (**), 0 flaws (~~), 3 warnings (==), 2 comments (--). Run idnits with the --verbose option for more detailed information about the items above. ----------------------------------------------------------------------- OPS-DIR mailing list OPS-DIR at ietf.org https://www.ietf.org/mailman/listinfo/ops-dir