[ARROW-10980] [Rust] Document use of unsafe in src/array/*.rs

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Invalid
• Component/s: Rust
• External issue URL: https://github.com/apache/arrow/issues/26903
• Assignee: Unassigned
• Reporter: Andy Grove
• Created: 19/Dec/20 15:40
• Updated: 11/Jan/23 08:16
• Resolved: 26/Apr/21 12:42

Description

This needs to be broken down into smaller tasks

 
There are ~32 uses of unsafe insrc/array/*.rs and we should document them according to the guidelines in the Arrow crate README

 

// JUSTIFICATION
//  Benefit
//      Describe the benefit of using unsafe. E.g.
//      "30% performance degradation if the safe counterpart is used, see bench X."
//  Soundness
//      Describe why the code remains sound (according to the definition of rust's unsafe code guidelines). E.g.
//      "We bounded check these values at initialization and the array is immutable."
let ... = unsafe { ... };