Skip to content

Conversation

@viirya
Copy link
Member

@viirya viirya commented Jan 23, 2024

Which issue does this PR close?

Closes #.

Rationale for this change

This is a follow up PR for the suggestion from @alamb (https://github.com/apache/arrow-datafusion/pull/8878/files#r1456452737) and update document.

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@github-actions github-actions bot added physical-expr Changes to the physical-expr crates optimizer Optimizer rules labels Jan 23, 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 for this PR @viirya

viirya and others added 2 commits January 23, 2024 12:12
Co-authored-by: Andrew Lamb <andrew@nerdnetworks.org>
@viirya viirya merged commit d6ab343 into apache:main Jan 24, 2024
@viirya
Copy link
Member Author

viirya commented Jan 24, 2024

Thanks @alamb

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

optimizer Optimizer rules physical-expr Changes to the physical-expr crates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants