Skip to content

Conversation

@findepi
Copy link
Member

@findepi findepi commented Oct 18, 2024

This makes the EXPLAIN of physical plan fuller. Previously it did not include type-related information.

Relates to #12997 . Maybe something related is needed for logical plans as well.

This makes the EXPLAIN of physical plan fuller. Previously it did not
include type-related information.
@github-actions github-actions bot added physical-expr Changes to the physical-expr crates sqllogictest SQL Logic Tests (.slt) labels Oct 18, 2024
@findepi findepi force-pushed the findepi/display-valuesexec-schema-in-explain-c88556 branch from 1ab8b66 to a519eb7 Compare October 18, 2024 11:44
Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

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

I think you can get the same effect with EXPLAIN verbose already:

> explain verbose select * from values (1);
....
|                                                            |                                                                                     |
| physical_plan_with_schema                                  | ValuesExec, schema=[column1:Int64;N]                                                |
|                                                            |                                                                                     |

@alamb alamb marked this pull request as draft October 18, 2024 20:29
@findepi findepi closed this Oct 18, 2024
@findepi findepi deleted the findepi/display-valuesexec-schema-in-explain-c88556 branch October 18, 2024 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

physical-expr Changes to the physical-expr crates sqllogictest SQL Logic Tests (.slt)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants