Skip to content

Commit

Permalink
Add the test value
Browse files Browse the repository at this point in the history
  • Loading branch information
blaginin committed Nov 19, 2024
1 parent 4005c0c commit 91b468d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 1 addition & 2 deletions datafusion/functions-nested/src/make_array.rs
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ use arrow_array::{
new_null_array, Array, ArrayRef, GenericListArray, NullArray, OffsetSizeTrait,
};
use arrow_buffer::OffsetBuffer;
use arrow_schema::DataType::{LargeList, List, Null};
use arrow_schema::DataType::{List, Null};
use arrow_schema::{DataType, Field};
use datafusion_common::{plan_err, utils::array_into_list_array_nullable, Result};
use datafusion_expr::binary::{
Expand Down Expand Up @@ -198,7 +198,6 @@ pub(crate) fn make_array_inner(arrays: &[ArrayRef]) -> Result<ArrayRef> {
let array = new_null_array(&DataType::Int64, length);
Ok(Arc::new(array_into_list_array_nullable(array)))
}
LargeList(..) => array_array::<i64>(arrays, data_type),
_ => array_array::<i32>(arrays, data_type),
}
}
Expand Down
2 changes: 2 additions & 0 deletions datafusion/sqllogictest/test_files/array.slt
Original file line number Diff line number Diff line change
Expand Up @@ -1158,6 +1158,8 @@ select column1, column5 from arrays_values_without_nulls;
# make array with arrays of different types
query ?
select make_array(make_array(1), arrow_cast(make_array(-1), 'LargeList(Int8)'))
----
[[1], [-1]]

query ???
select make_array(column1),
Expand Down

0 comments on commit 91b468d

Please sign in to comment.