kszlim commented on code in PR #6595:
URL: https://github.com/apache/arrow-rs/pull/6595#discussion_r1807967283
##########
arrow-array/src/builder/struct_builder.rs:
##########
@@ -253,6 +253,13 @@ pub fn make_builder(datatype: &DataType, capacity: usize)
-> Box<dyn ArrayBuilde
let builder = make_builder(field.data_type(), capacity);
Box::new(LargeListBuilder::with_capacity(builder,
capacity).with_field(field.clone()))
}
+ DataType::FixedSizeList(field, size) => {
+ let builder = make_builder(field.data_type(), capacity);
Review Comment:
Hmm are you sure, @tustvold I followed the pattern for the above
Listbuilder, seems like either that's wrong too or this is right.
--
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]
