-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Remove the query_analyzer
binary
#14054
Comments
frouioui
added
Type: Feature Request
Needs Triage
This issue needs to be correctly labelled and triaged
Type: Internal Cleanup
Component: Query Serving
and removed
Type: Feature Request
Needs Triage
This issue needs to be correctly labelled and triaged
labels
Sep 20, 2023
4 tasks
ajm188
pushed a commit
to planetscale/vitess
that referenced
this issue
Sep 20, 2023
closes vitessio#14054 Signed-off-by: Andrew Mason <andrew@planetscale.com>
4 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Feature Description
I have found that the
query_analyzer
is not used throughout the codebase and the only commits made to it in the recent years were maintenance commits. I think it is fine to remove this binary.Use Case(s)
Removing it will give us lighter release packages and less code to maintain.
The text was updated successfully, but these errors were encountered: