-
Notifications
You must be signed in to change notification settings - Fork 1.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.
Part of #5374
@XiangpengHao implemented row format --> ByteView (StringView / BinaryView) encoding/decoing in https://github.com/apache/arrow-rs/pull/5922/files
Describe the solution you'd like
- Add a benchmark for StringView to https://github.com/apache/arrow-rs/blob/master/arrow/benches/row_format.rs
- Make the row conversion faster
Describe alternatives you've considered
@tustvold noted that a potential performance improvement https://github.com/apache/arrow-rs/pull/5922/files#r1650032068 -- namely to build the array directly rather than build a StringArray and then convert to StringView
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