Jefffrey commented on code in PR #9376:
URL: https://github.com/apache/arrow-rs/pull/9376#discussion_r2785296097
##########
arrow-array/src/array/run_array.rs:
##########
@@ -123,6 +123,28 @@ impl<R: RunEndIndexType> RunArray<R> {
Ok(array_data.into())
}
+ /// Create a new [`RunArray`] from the provided parts, without validation
+ ///
+ /// # Safety
+ ///
+ /// Safe if [`Self::try_new`] would not error
+ pub unsafe fn new_unchecked(
+ data_type: DataType,
+ run_ends: RunEndBuffer<R::Native>,
+ values: ArrayRef,
+ ) -> Self {
Review Comment:
Do we need to add force validate here like for other arrays
https://github.com/apache/arrow-rs/blob/765c5b715ddf5be605169105e3e65429e107713d/arrow-array/src/array/dictionary_array.rs#L332-L336
https://github.com/apache/arrow-rs/blob/765c5b715ddf5be605169105e3e65429e107713d/arrow-array/src/array/byte_array.rs#L162-L171
--
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]
