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

Remove vtqueryserver #5583

Merged
merged 2 commits into from
Dec 21, 2019
Merged

Conversation

morgo
Copy link
Contributor

@morgo morgo commented Dec 17, 2019

Fixes #5531

This PR proposes that we remove vtqueryserver, which per this discussion is not being used.

The tool was never documented on vitess.io, so my understanding is that there is no current usage. But if reviewers feel otherwise, I am happy to change the PR to remove the tests and then add a note to the 5.0 release notes saying it is deprecated.

The motivation for removing this, is that it was failing unit test when using MariaDB as the mysql flavor. It is not the only test that breaks, but it is easier to remove rather than fix it. I originally had this in the PR #5559 ; but have now forked it to a separate PR.

Signed-off-by: Morgan Tocker [email protected]

Signed-off-by: Morgan Tocker <[email protected]>
@morgo morgo requested a review from sougou as a code owner December 17, 2019 19:46
@derekperkins
Copy link
Member

Kill it

@guidoiaquinti
Copy link
Member

cc @demmer

@sougou
Copy link
Contributor

sougou commented Dec 20, 2019

Need to resolve conflicts.

@demmer
Copy link
Member

demmer commented Dec 20, 2019

👍 I'm ok to remove this.

It was an interesting experiment originally, but I agree it's not worth the code bloat to keep it around. If a use case comes up again we can resurrect.

@sougou sougou merged commit 1dc9150 into vitessio:master Dec 21, 2019
@morgo morgo deleted the morgo-rm-vtqueryserver branch December 21, 2019 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

failing unit test: vtqueryserver on debian10/mariadb 10.3
5 participants