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

Routing of Information Schema Queries #7841

Merged
merged 5 commits into from
Apr 21, 2021

Conversation

GuptaManan100
Copy link
Member

Description

This Pull Request fixes the routing regarding Information Schema Queries. More information schema queries can now be merged correctly. SysTableTableSchema and SysTableTableName are now made lists and the error is thrown from the executor instead of the planner.

Related Issue(s)

Checklist

  • Should this PR be backported?
  • Tests were added or are not required
  • Documentation was added or is not required

Impacted Areas in Vitess

Components that this PR will affect:

  • Query Serving
  • VReplication
  • Cluster Management
  • Build/CI
  • VTAdmin

Signed-off-by: GuptaManan100 <manan@planetscale.com>
Signed-off-by: GuptaManan100 <manan@planetscale.com>
Signed-off-by: GuptaManan100 <manan@planetscale.com>
Signed-off-by: Harshit Gangal <harshit@planetscale.com>
go/test/endtoend/vtgate/system_schema_test.go Outdated Show resolved Hide resolved
go/vt/vtgate/engine/route.go Outdated Show resolved Hide resolved
Signed-off-by: GuptaManan100 <manan@planetscale.com>
@harshit-gangal harshit-gangal merged commit 726955e into vitessio:master Apr 21, 2021
@harshit-gangal harshit-gangal deleted the information_schema_join branch April 21, 2021 05:12
@systay systay mentioned this pull request Apr 23, 2021
systay pushed a commit to planetscale/vitess that referenced this pull request May 11, 2021
Backport of vitessio#7841

* made sysTableTableSchema a slice and fixed merging for SelectDBA queries
* updated tests
* added tests
* changed error message

Signed-off-by: GuptaManan100 <manan@planetscale.com>

Signed-off-by: Andres Taylor <andres@planetscale.com>
rafael pushed a commit to tinyspeck/vitess that referenced this pull request May 11, 2021
Backport of vitessio#7841

* made sysTableTableSchema a slice and fixed merging for SelectDBA queries
* updated tests
* added tests
* changed error message

Signed-off-by: GuptaManan100 <manan@planetscale.com>

Signed-off-by: Andres Taylor <andres@planetscale.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect Output from Information Schema Query
2 participants