Skip to content

Conversation

@lewiszlw
Copy link
Member

@lewiszlw lewiszlw commented Aug 1, 2024

Which issue does this PR close?

Closes #.

Rationale for this change

When I was debugging optimizer rule, I found many logical plan nodes are not derived Debug, so added it.

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@github-actions github-actions bot added the logical-expr Logical plan and expressions label Aug 1, 2024
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.

Thank you @lewiszlw -- this looks great to me

@alamb alamb merged commit 6e2ff29 into apache:main Aug 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

logical-expr Logical plan and expressions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants