Re: [quicwg/base-drafts] Congestion window reduced on ACK (#2244)

[ianswett <[e-mail]>]

ianswett approved this pull request.

Thanks for fixing this.  Some editorial suggestions, but LG otherwise.

> @@ -1023,15 +1027,13 @@ kLossReductionFactor:
   The RECOMMENDED value is 0.5.
 
 kPersistentCongestionThreshold:
-: Number of consecutive PTOs after which network is considered to be
+: Number of consecutive PTOs after which network might be considered to be

```suggestion
: Number of consecutive PTOs after which the network is considered to be
```

I prefer "is" instead of "might be" given you already have considered.

>    experiencing persistent congestion.  The rationale for this threshold is to
-  enable a sender to use initial PTOs for aggressive probing, similar to Tail
-  Loss Probe (TLP) in TCP {{TLP}} {{RACK}}.  Once the number of consecutive PTOs
-  reaches this threshold - that is, persistent congestion is established - the
-  sender responds by collapsing its congestion window to kMinimumWindow, similar
-  to a Retransmission Timeout (RTO) in TCP {{RFC5681}}.  The RECOMMENDED value
-  for kPersistentCongestionThreshold is 2, which is equivalent to having two
-  TLPs before an RTO in TCP.
+  enable a sender to use initial PTOs for aggressive probing, as TCP does with
+  Tail Loss Probe (TLP) {{TLP}} {{RACK}}, before establishing persistent
+  congestion, as TCP does with a Retransmission Timeout (RTO) {{?RFC5681}}.  The
+  RECOMMENDED value for kPersistentCongestionThreshold is 2, which is equivalent
+  to having two TLPs before an RTO in TCP.

```suggestion
  to two TLPs before an RTO in TCP.
```

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/quicwg/base-drafts/pull/2244#pullrequestreview-187538518