Skip to content

Add EXPLAIN format for Vitess plans#6110

Merged
harshit-gangal merged 12 commits intovitessio:masterfrom
systay:explain-take-2
Apr 28, 2020
Merged

Add EXPLAIN format for Vitess plans#6110
harshit-gangal merged 12 commits intovitessio:masterfrom
systay:explain-take-2

Conversation

@systay
Copy link
Copy Markdown
Collaborator

@systay systay commented Apr 25, 2020

This PR adds a new format option for EXPLAIN that will show the built Vitess plan

EXPLAIN FORMAT=VITESS <QUERY>

@systay systay requested a review from harshit-gangal April 25, 2020 15:31
@systay systay requested a review from sougou as a code owner April 25, 2020 15:31
@systay systay force-pushed the explain-take-2 branch 3 times, most recently from 0343dfd to dda586a Compare April 28, 2020 06:07
systay added 11 commits April 28, 2020 08:08
Signed-off-by: Andres Taylor <andres@planetscale.com>
Signed-off-by: Andres Taylor <andres@planetscale.com>
Signed-off-by: Andres Taylor <andres@planetscale.com>
Signed-off-by: Andres Taylor <andres@planetscale.com>
Signed-off-by: Andres Taylor <andres@planetscale.com>
Signed-off-by: Andres Taylor <andres@planetscale.com>
Signed-off-by: Andres Taylor <andres@planetscale.com>
Signed-off-by: Andres Taylor <andres@planetscale.com>
Signed-off-by: Andres Taylor <andres@planetscale.com>
Signed-off-by: Andres Taylor <andres@planetscale.com>
Signed-off-by: Andres Taylor <andres@planetscale.com>
}, {
input: "desc foobar",
input: "explain t1",
output: "otherread",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add these test back.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Signed-off-by: Andres Taylor <andres@planetscale.com>
@harshit-gangal harshit-gangal merged commit c2ba731 into vitessio:master Apr 28, 2020
@deepthi deepthi added this to the v7.0 milestone Jul 27, 2020
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.

3 participants