Skip to content
This repository has been archived by the owner on Aug 28, 2023. It is now read-only.

Add a variant of the text Gorry proposed #145

Merged
merged 9 commits into from
May 30, 2022
Merged

Add a variant of the text Gorry proposed #145

merged 9 commits into from
May 30, 2022

Conversation

larseggert
Copy link
Contributor

Copy link
Contributor

@nealcardwell nealcardwell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, Lars. Looks good to me, FWIW. I noticed one typo, which I commented on in-line.

draft-ietf-tcpm-rfc8312bis.md Outdated Show resolved Hide resolved
Copy link
Contributor

@lisongxu lisongxu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

draft-ietf-tcpm-rfc8312bis.md Outdated Show resolved Hide resolved
draft-ietf-tcpm-rfc8312bis.md Outdated Show resolved Hide resolved
draft-ietf-tcpm-rfc8312bis.md Outdated Show resolved Hide resolved
draft-ietf-tcpm-rfc8312bis.md Outdated Show resolved Hide resolved
draft-ietf-tcpm-rfc8312bis.md Outdated Show resolved Hide resolved
Also, the wide deployment of CUBIC on the Internet was driven by
direct adoption in most of the popular operating systems, and did not
follow the practices documented in {{!RFC5033}}. However, due to the
resulting extremely widespread deployment experience over a long
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd suggest removing /extremely/ ... it might be true that this is extreme, but all that matters is the widespread deployment experience.

Copy link
Contributor Author

@larseggert larseggert May 25, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about "Internet-scale" instead of "extremely widespread"?

Copy link
Contributor

@gorryfair gorryfair left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this text captures what I hoped for. With the suggestion to remove /extremely/, I think this text is ready.

@larseggert larseggert merged commit 8d29d99 into main May 30, 2022
@larseggert larseggert deleted the gorrys-text branch May 30, 2022 08:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants