tustvold commented on code in PR #4381:
URL: https://github.com/apache/arrow-rs/pull/4381#discussion_r1222903745


##########
arrow-array/src/array/list_array.rs:
##########
@@ -57,6 +57,8 @@ impl OffsetSizeTrait for i64 {
 /// An array of [variable length 
arrays](https://arrow.apache.org/docs/format/Columnar.html#variable-size-list-layout)
 ///
 /// See [`ListArray`] and [`LargeListArray`]`
+///
+/// See [`GenericListBuilder`](crate::builder::GenericListBuilder) for how to 
construct a [`GenericListArray`]

Review Comment:
   For the more complex types I opted to just point people to the builders, 
whilst the arrays can be constructed in other ways these necessarily require 
knowledge of how these types are structured, or are limited to a specific 
subset. I think it is better to just encourage people to use the builders as a 
first port of call



-- 
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]

Reply via email to