This is an automated email from the ASF dual-hosted git repository.
Jefffrey pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow-rs.git
The following commit(s) were added to refs/heads/main by this push:
new e1e71ea89b Fix BinaryViewArray construction after buffer ownership API
change (#10814)
e1e71ea89b is described below
commit e1e71ea89b211758e20254296008f96e744ed138
Author: Yiming Qiao <[email protected]>
AuthorDate: Mon Aug 24 15:07:52 2026 +0800
Fix BinaryViewArray construction after buffer ownership API change (#10814)
Fixes the parquet-variant-compute compilation failure introduced when
#10708 was merged after #10640.
BinaryViewArray::new_unchecked now expects Arc<[Buffer]>, but the call
added by #10640 still passed Vec<Buffer>. This converts the newly
constructed buffer vector into the required shared slice.
Verified with:
- cargo fmt --all -- --check
- cargo check -p parquet-variant-compute --all-targets
---
parquet-variant-compute/src/variant_array_builder.rs | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/parquet-variant-compute/src/variant_array_builder.rs
b/parquet-variant-compute/src/variant_array_builder.rs
index d7bcd08937..92efaa640a 100644
--- a/parquet-variant-compute/src/variant_array_builder.rs
+++ b/parquet-variant-compute/src/variant_array_builder.rs
@@ -474,7 +474,7 @@ fn binary_view_array_from_buffers(buffer: Vec<u8>, offsets:
Vec<usize>) -> Binar
// SAFETY: `make_view` constructs every view from an in-bounds slice of
buffer 0, and there
// are no nulls. The buffer length check above guarantees every offset
fits in a `u32`.
- unsafe { BinaryViewArray::new_unchecked(views.into(), vec![buffer], None) }
+ unsafe { BinaryViewArray::new_unchecked(views.into(), vec![buffer].into(),
None) }
}
#[cfg(test)]
