brunal opened a new issue, #9459:
URL: https://github.com/apache/arrow-rs/issues/9459

   TypedRunArray dfn is simply:
   
   ```
   pub struct TypedRunArray<'a, R: RunEndIndexType, V> {
       /// The run array
       run_array: &'a RunArray<R>,
   
       /// The values of the run_array
       values: &'a V,
   }
   ```
   
   Strangely enough, there's no constraint about `V`. But in practice, `V: 
Array`.
   
   Impls involving TypedRunArray mandate that `&'a V: ArrayAccessor`.
   
   **Describe the solution you'd like**
   
   Add either `V: Array` or `where &'a V: ArrayAccessor` in `TypedRunArray` 
struct definition. I'm not sure which is best. The former is stronger, the 
latter is more aligned with usages of TypedRunArray in the codebase.
   
   The former requires tweaking many uses of `TypedRunArray` to add the bound, 
the latter changes nothing (as the bound is currently covered).
   
   **Describe alternatives you've considered**
   
   Keep it as-is.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to