Skip to content
This repository has been archived by the owner on Oct 11, 2024. It is now read-only.

Server controlled retry delay #1040

Open
gdbelvin opened this issue Sep 17, 2018 · 2 comments
Open

Server controlled retry delay #1040

gdbelvin opened this issue Sep 17, 2018 · 2 comments

Comments

@gdbelvin
Copy link
Contributor

Replace MinInterval with a server controlled retry delay.

This will allow the server to dynamically and organically rate limit traffic when the server is under load and the queue of mutations is long.

@Zyqsempai
Copy link
Contributor

Sounds interesting, can I help somehow?

@gdbelvin
Copy link
Contributor Author

gdbelvin commented Jul 16, 2019

Yes, this is important. (And thank you for all of your prior contributions!)

Clients currently use a static duration for controlling how often to enqueue new mutations while they are watching for new updates. If the sequencer gets behind and the queue of mutations grows, clients will pile on more and more mutations exasperating the problem.

The ideal solution would be to have the QueueMutation API return an indicator of how long the queue of mutations currently is and have clients wait that amount of time + jitter.

The sequencer currently reports the depth of the queue behind the UpdateMetrics API. Perhaps the frontend could periodically call this and cache the results?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants