-
Notifications
You must be signed in to change notification settings - Fork 1k
Closed
Labels
arrowChanges to the arrow crateChanges to the arrow crateenhancementAny new improvement worthy of a entry in the changelogAny new improvement worthy of a entry in the changelog
Description
Is your feature request related to a problem or challenge? Please describe what you are trying to do.
Current min/max support was added in #6053, with a template solution for generic strings. This is suboptimal, as string view can benefit from faster min/max just like it can have faster cmp
operations.
Describe the solution you'd like
Use the fast string view cmp operation instead of the generic one. My local testing shows that a special implementation can be twice as fast.
Part of #5374, and apache/datafusion#10918
Describe alternatives you've considered
Additional context
Metadata
Metadata
Assignees
Labels
arrowChanges to the arrow crateChanges to the arrow crateenhancementAny new improvement worthy of a entry in the changelogAny new improvement worthy of a entry in the changelog