This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/datafusion.git


The following commit(s) were added to refs/heads/main by this push:
     new 8833253d7b fix: map benchmark failing (#18469)
8833253d7b is described below

commit 8833253d7bce3d9da1c1a6484373b79a63deb00f
Author: Randy <[email protected]>
AuthorDate: Tue Nov 4 10:02:46 2025 +0800

    fix: map benchmark failing (#18469)
    
    ##  Which issue does this PR close?
    
    - map benchmark failing #18454
    
    ## Rationale for this change
    
    - Map expects list, large_list or fixed_size_list but Scalar type is
    provided
    
    ## What changes are included in this PR?
    
    - Change the types to the types of keys and values
    
    ## Are these changes tested?
    
    - Previously failing tests pass
    
    ## Are there any user-facing changes?
    
    N/A
---
 datafusion/functions-nested/benches/map.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/datafusion/functions-nested/benches/map.rs 
b/datafusion/functions-nested/benches/map.rs
index ca12dde1f5..16bdbd340a 100644
--- a/datafusion/functions-nested/benches/map.rs
+++ b/datafusion/functions-nested/benches/map.rs
@@ -98,7 +98,7 @@ fn criterion_benchmark(c: &mut Criterion) {
         let values = 
ColumnarValue::Scalar(ScalarValue::List(Arc::new(value_list)));
 
         let return_type = map_udf()
-            .return_type(&[DataType::Utf8, DataType::Int32])
+            .return_type(&[keys.data_type(), values.data_type()])
             .expect("should get return type");
         let arg_fields = vec![
             Field::new("a", keys.data_type(), true).into(),


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to