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 49a04334ba fix: New bench added to  `coalesce_kernels` bench dies due 
to wrong field nullability (#11126)
49a04334ba is described below

commit 49a04334baa47430e8b9efb50f7742a9ba1ad137
Author: Ed Seidl <[email protected]>
AuthorDate: Thu Sep 17 17:33:22 2026 -0700

    fix: New bench added to  `coalesce_kernels` bench dies due to wrong field 
nullability (#11126)
    
    # Which issue does this PR close?
    N/A
    
    # Rationale for this change
    
    #11123 introduced some new benches, one of which fails with
    ```
    called `Result::unwrap()` on an `Err` value: InvalidArgumentError("column 
types must match schema types, expected FixedSizeList(4 x Int32) but found 
FixedSizeList(4 x non-null Int32) at column index 0")
    ```
    
    # What changes are included in this PR?
    
    Change the nullability for the FSL fields to not nullable
    
    # Are these changes tested?
    N/A
    
    # Are there any user-facing changes?
    
    Benches now run.
---
 arrow/benches/coalesce_kernels.rs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arrow/benches/coalesce_kernels.rs 
b/arrow/benches/coalesce_kernels.rs
index b455b2d7d3..dca87ea6cc 100644
--- a/arrow/benches/coalesce_kernels.rs
+++ b/arrow/benches/coalesce_kernels.rs
@@ -124,7 +124,7 @@ fn add_all_filter_benchmarks(c: &mut Criterion) {
 
     let single_fsl4_schema = SchemaRef::new(Schema::new(vec![Field::new(
         "value",
-        DataType::FixedSizeList(Arc::new(Field::new("item", DataType::Int32, 
true)), 4),
+        DataType::FixedSizeList(Arc::new(Field::new("item", DataType::Int32, 
false)), 4),
         true,
     )]));
 
@@ -464,7 +464,7 @@ fn add_all_take_benchmarks(c: &mut Criterion) {
 
     let single_fsl4_schema = SchemaRef::new(Schema::new(vec![Field::new(
         "value",
-        DataType::FixedSizeList(Arc::new(Field::new("item", DataType::Int32, 
true)), 4),
+        DataType::FixedSizeList(Arc::new(Field::new("item", DataType::Int32, 
false)), 4),
         true,
     )]));
 

Reply via email to