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

Recovery-31: Congestion control based on ACKs only to speed up? #4194

Closed
gloinul opened this issue Oct 12, 2020 · 4 comments
Closed

Recovery-31: Congestion control based on ACKs only to speed up? #4194

gloinul opened this issue Oct 12, 2020 · 4 comments
Labels
-recovery ietf-lc An issue that was raised during IETF Last Call.

Comments

@gloinul
Copy link
Contributor

gloinul commented Oct 12, 2020

Section 7.

"Unlike TCP, QUIC can detect the loss of these packets and MAY use that information to adjust the congestion controller or the rate of ACK-only packets being sent, but this document does not describe a mechanism for doing so."

So is there need to say anything that if one does this it needs to be adjusted in both directions, i.e. it is not okay to only use it to speed up, and not slow down.

@ianswett
Copy link
Contributor

As it stands, the statement uses the neutral word 'adjust' which does not imply up or down. I think this is appropriate and given this is more of a research project, I'd prefer to keep the text approximately as is.

@larseggert larseggert added the ietf-lc An issue that was raised during IETF Last Call. label Oct 13, 2020
@larseggert larseggert added this to Triage in Late Stage Processing via automation Oct 13, 2020
@janaiyengar
Copy link
Contributor

I agree with @ianswett. The sentence actually says what you want it to say, in that it doesn't specify a direction of adjustment.

@larseggert
Copy link
Member

@gloinul do these explanations address your issue? Or are text changes required?

@gloinul
Copy link
Contributor Author

gloinul commented Oct 15, 2020

Yes, lets close this issue.

Late Stage Processing automation moved this from Triage to Issue Handled Oct 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-recovery ietf-lc An issue that was raised during IETF Last Call.
Projects
Late Stage Processing
  
Issue Handled
Development

No branches or pull requests

4 participants