Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

I am concerned the congestion control text is too permissive. #3247

Closed
gorryfair opened this issue Nov 17, 2019 · 2 comments · Fixed by #3248
Closed

I am concerned the congestion control text is too permissive. #3247

gorryfair opened this issue Nov 17, 2019 · 2 comments · Fixed by #3248
Labels
-recovery design An issue that affects the design of the protocol; resolution requires consensus. has-consensus An issue that the Chairs have determined has consensus, by canvassing the mailing list.

Comments

@gorryfair
Copy link
Contributor

The docs call out that the design is based on (new)Reno. That is good and suitable for a PS. I have no problem here.

However, there are several places where the text is very permissive "MAY..." that suggests the PS can implement this or anything else. This combination seems awkward for a PS. I'd hoped that we could allow experimentation with new CC's but say SHOULD use an IETF-specified congestion control method, or something text close to that, and change the MAYs into something more helpful. I think this is an easy piece of text to get correct.

@gorryfair
Copy link
Contributor Author

Please check appendices, e.g.

" Constants used in loss recovery are based on a combination of RFCs,
papers, and common practice. Some may need to be changed or
negotiated in order to better suit a variety of environments."

  • that final sentence needs thought.

@janaiyengar janaiyengar added -recovery design An issue that affects the design of the protocol; resolution requires consensus. labels Nov 18, 2019
@janaiyengar janaiyengar removed the design An issue that affects the design of the protocol; resolution requires consensus. label Nov 25, 2019
@janaiyengar
Copy link
Contributor

If this goes into late-stage, it should be design.

@mnot mnot added the design An issue that affects the design of the protocol; resolution requires consensus. label Nov 25, 2019
@ianswett ianswett added the proposal-ready An issue which has a proposal that is believed to be ready for a consensus call. label Dec 10, 2019
@mnot mnot added has-consensus An issue that the Chairs have determined has consensus, by canvassing the mailing list. and removed proposal-ready An issue which has a proposal that is believed to be ready for a consensus call. labels Jan 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-recovery design An issue that affects the design of the protocol; resolution requires consensus. has-consensus An issue that the Chairs have determined has consensus, by canvassing the mailing list.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants