[ARROW-9916] [RUST] Avoid cloning ArrayData in several places

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.0.0
• Fix Version/s: 2.0.0
• Component/s: Rust
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/25948
• Assignee: Jörn Horstmann
• Reporter: Jörn Horstmann
• Created: 04/Sep/20 19:17
• Updated: 11/Jan/23 11:09
• Resolved: 05/Sep/20 20:03

Description

I noticed this while benchmarking improvements in ARROW-9895. A flamegraph showed a significant amount of time spent in Arc::clone/atomic_add followed by Arc::drop/atomic_sub

The Array trait has two methods for accessing ArrayData, `.data()` which clones an `Arc<ArrayData>` and `.data_ref()` which only borrows the data. In many places borrow can be used instead of clone.

Issue Links

• links to: GitHub Pull Request #8113