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

Feature Request: Add the ability to skip a list of tablets to pick in the TabletPicker #14097

Closed
pbibra opened this issue Sep 26, 2023 · 0 comments · Fixed by #14224
Closed

Comments

@pbibra
Copy link
Contributor

pbibra commented Sep 26, 2023

Feature Description

There are certain cases during a VTGate VStream where we pick a tablet, the stream fails and we exit immediately before retrying another tablet. An example of this are GTID mismatch errors. In these cases, the tablet is healthy so a client retry may choose the same one (especially if there's only one tablet in the same cell as the vtgate as we have a local cell preference by default) and fail on a GTID mismatch again. Instead, we'd like to retry with a list of tablets to omit from the viable candidates that the TabletPicker chooses from.

Use Case(s)

More resilient VStream RPCs that effectively retry when appropriate.

For example, GTID mismatch errors, un-healthy, or NOT_FOUND tablet errors for Vitess version that will not have this fix: #13582

@pbibra pbibra added Needs Triage This issue needs to be correctly labelled and triaged Type: Feature labels Sep 26, 2023
@deepthi deepthi added Component: VReplication and removed Needs Triage This issue needs to be correctly labelled and triaged labels Oct 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants