-
Notifications
You must be signed in to change notification settings - Fork 451
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
EXPLAIN(DISTSQL,TYPES) #7045
EXPLAIN(DISTSQL,TYPES) #7045
Conversation
93c7f21
to
c88432b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note that on 20.1 new explain-analyze-distsql-plan.png
image has the diagram of the vectorized plan. Now that we decided to preserve 19.2 behavior with regards to vectorize
setting, I don't think we want the change to the diagram.
Reviewed 8 of 8 files at r1.
Reviewable status:complete! 0 of 0 LGTMs obtained
c88432b
to
8f6c072
Compare
@yuzefovich Got it. Thanks. I updated the PR. Re: "we decided to preserve 19.2 behavior with regards to vectorize setting", we should follow up about #7007 offline. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewable status:
complete! 1 of 0 LGTMs obtained (waiting on @jseldess)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewable status:
complete! 2 of 0 LGTMs obtained (waiting on @jseldess)
Fixes #6447.
EXPLAIN(DISTSQL,TYPES)
and example statement and diagram toEXPLAIN
page.EXPLAIN ANALYZE
page.