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

Clarify Use of Application Congestion Control #625

Closed
martinduke opened this issue Aug 7, 2020 · 2 comments · Fixed by #651
Closed

Clarify Use of Application Congestion Control #625

martinduke opened this issue Aug 7, 2020 · 2 comments · Fixed by #651

Comments

@martinduke
Copy link
Contributor

sec 5.2.9
It would be helpful to say "Applications that neither prohibit nor require congestion should query the result and disable their own congestion control if present in the transport"

@gorryfair
Copy link
Contributor

Interesting, but we need to be careful in what we say. It seems safe, but not performant to do CC at multiple layers, and if it's done at a lower layer performance might be improved by not replicating the function at a higher layer...

@tfpauly
Copy link
Contributor

tfpauly commented Aug 14, 2020

Link to UDP guidelines document.

Just mention that there are performance implications of multiple CC loops; don't just say "disable CC".

mwelzl added a commit that referenced this issue Sep 9, 2020
Closes #625, #627, #629. Partially addresses #631.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants