Dandandan commented on a change in pull request #9469:
URL: https://github.com/apache/arrow/pull/9469#discussion_r574657328
##########
File path: rust/datafusion/src/physical_plan/parquet.rs
##########
@@ -665,7 +661,7 @@ fn build_statistics_array(
statistics
} else {
// no row group has statistics defined
- return build_null_array(data_type, statistics_count);
+ return new_array_with_nulls(data_type, statistics_count);
Review comment:
Not sure whether you want to change it in this PR, but there is some
code that could use `new_array_with_nulls` too in the function
`to_array_of_size`, which uses this:
`Arc::new(repeat(None).take(size).collect::<Float64Array>())`
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
