Skip to content

Conversation

@edmondop
Copy link
Contributor

@edmondop edmondop commented Jul 3, 2024

Closes #11152 which I encountered when dealing with #10943

@github-actions github-actions bot added the physical-expr Changes to the physical-expr crates label Jul 3, 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.

Looks like a good step to me -- thanks @edmondop

I think the next step will me to add an equivalent to AggregateUDFImpl

@alamb alamb merged commit 1ffe053 into apache:main Jul 3, 2024
comphead pushed a commit to comphead/arrow-datafusion that referenced this pull request Jul 8, 2024
findepi pushed a commit to findepi/datafusion that referenced this pull request Jul 16, 2024
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove Min/Max references from AggregateExec::get_minmax_descr

2 participants