[ARROW-10903] [Rust] Implement FromIter<Option<Vec<u8>>> constructor for FixedSizeBinaryArray

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 4.0.0
• Component/s: Rust
• Labels: beginner beginner-friendly pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/26835
• Assignee: Ivan Vankov
• Reporter: Andrew Lamb
• Created: 14/Dec/20 18:53
• Updated: 11/Jan/23 08:16
• Resolved: 18/Mar/21 13:42

Description

Context: https://github.com/apache/arrow/pull/8853#discussion_r542189448 suggestion by jorgecarleitao

Basically, the goal would be to remove

impl From<Vec<Option<Vec<u8>>>> for FixedSizeBinaryArray {

Which while mostly used in tests, panics in certain situations.

Goal would be to replace that implementation with a `FromIter` implementation which is more performance, more general, and has the same ergonomics (from(vec![].into_iter()) instead of from(vec![...]) for a vector).

Issue Links

• links to: GitHub Pull Request #9647