tustvold commented on code in PR #4226:
URL: https://github.com/apache/arrow-rs/pull/4226#discussion_r1200043530
##########
parquet/src/arrow/array_reader/list_array.rs:
##########
@@ -227,8 +228,13 @@ impl<OffsetSize: OffsetSizeTrait> ArrayReader for
ListArrayReader<OffsetSize> {
let list_data = unsafe { data_builder.build_unchecked() };
- let result_array = GenericListArray::<OffsetSize>::from(list_data);
- Ok(Arc::new(result_array))
+ let result_array: ArrayRef = match *list_data.data_type() {
+ ArrowType::FixedSizeList(_, _) => {
+ Arc::new(FixedSizeListArray::from(list_data))
Review Comment:
This is actually significantly more complicated, as the null padding logic
is different for fixed size types. I'll have a think about how this could be
supported
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
