-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Closed as not planned
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 changelogperformance
Description
Is your feature request related to a problem or challenge? Please describe what you are trying to do.
@Dandandan noted on https://github.com/apache/arrow-rs/pull/7513/files#r2119136807:
Looking at the code - this doesn't actually really use the TrustedLen optimization.
I think it could use the same optimization as MutableBuffer::from_trusted_len
Describe the solution you'd like
Let's optimize that function!
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 changelogperformance