Telechat Review of draft-ietf-rtcweb-fec-08
review-ietf-rtcweb-fec-08-opsdir-telechat-schoenwaelder-2019-02-12-00

Request Review of draft-ietf-rtcweb-fec
Requested rev. no specific revision
Type Telechat Review
Team Ops Directorate (opsdir)
Deadline 2019-02-19
Requested 2019-02-07
Other Reviews Secdir Last Call review of -08 by Hilarie Orman
Review State Completed
Reviewer Jürgen Schönwälder
Review review-ietf-rtcweb-fec-08-opsdir-telechat-schoenwaelder-2019-02-12
Posted at https://mailarchive.ietf.org/arch/msg/ops-dir/Ye76EC6B27fviMFvjsurd_LpGD0
Reviewed rev. 08
Review result Has Nits
Draft last updated 2019-02-12
Review completed: 2019-02-12

Review
review-ietf-rtcweb-fec-08-opsdir-telechat-schoenwaelder-2019-02-12

I have reviewed this document and I did not find any issues that are
affect the operations of networks (except that FEC requires more
bandwidth and that it may not help much or even make things worse if
bandwidth is the cause of packet loss, which is explained in the
document).

While reading the document, I wrote down the following notes that the
editor may take into account when revising the document:

- in 3.1, expland SSRC on first usage and say that this is about
  sending streams over RTP somewhere early on (I assume this is
  implied by using the term WebRTC but for outsiders like me it may
  help to be more specific).

- To what extend is this document WebRTC specific? Do the requirements
  also apply if I use RTP without a WebRTC context? If so, should the
  title rather say "RTP Forward Error Correction Requirements"? Well,
  section 6 may be WebRTC specific but that section just says that
  nothing is being recommended, so the recommendations are really all
  about FEC usage over RTP as far as I can tell (as an outsider).

- stylistic: I am not a big fan of using citations like '[RFC2198]' as
  ordinary words or nouns, it makes text difficult to follow unless
  you know the RFC numbers by heart and your brain translates them
  back to something meaningful on the fly. This makes texts more
  difficult to read for outsiders. Example:

     This mechanism is similar to the
     [RFC2198] mechanism described above.

  I prefer this:

     This mechanism is similar to the
     redundant encoding mechanism described above.

  There	are couple of such usages of [RFCXXXX] in the document

- add reference for PCMU