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

Pull Queries: Local host should immediately know lags and fail stale requests #4506

Closed
AlanConfluent opened this issue Feb 10, 2020 · 0 comments · Fixed by #4597
Closed

Pull Queries: Local host should immediately know lags and fail stale requests #4506

AlanConfluent opened this issue Feb 10, 2020 · 0 comments · Fixed by #4597
Assignees
Labels
enhancement P0 Denotes must-have for a given milestone
Milestone

Comments

@AlanConfluent
Copy link
Member

Is your feature request related to a problem? Please describe.
If A is an active/standby for some table and B wants to forward requests to A, B currently relies on lag reporting to know how far behind A is. We currently allow B to issue requests to A during startup time, before A has reported its lag so as not to incur large error spikes during rolling bounces of hosts.

Unless we implement #4472 , so that B can do the enforcement in these situations, A should do maximum lag enforcement itself since it should always know its local lags.

Describe the solution you'd like
Host A, in the above description, should enforce maximum lags on local data and should fail pull queries if the data is too stale.

@vpapavas vpapavas added this to the 0.8.0 milestone Feb 12, 2020
@agavra agavra modified the milestones: 0.8.0, 0.7.0 Feb 12, 2020
@vpapavas vpapavas modified the milestones: 0.7.0, 0.8.0 Feb 12, 2020
@vinothchandar vinothchandar added the P0 Denotes must-have for a given milestone label Feb 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement P0 Denotes must-have for a given milestone
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants