sdf-jkl commented on code in PR #10152:
URL: https://github.com/apache/arrow-rs/pull/10152#discussion_r3438743332
##########
parquet-variant/src/builder.rs:
##########
@@ -477,6 +477,25 @@ impl<S: BuilderSpecificState> Drop for ParentState<'_, S> {
/// Top level builder for [`Variant`] values
///
+/// `VariantBuilder` builds a single, self-contained [`Variant`] value --
useful
+/// for one-off values and unit tests. To build an array (column) of variants,
+/// one per input row, use [`VariantArrayBuilder`] from the
+/// `parquet-variant-compute` crate rather than a `VariantBuilder` per row. It
+/// implements [`VariantBuilderExt`], so you append values and nested objects
or
+/// lists the same way:
+///
+/// ```ignore
Review Comment:
https://github.com/apache/arrow-rs/pull/10152/commits/964506df95dc461372404df751de98c37fc6ab88
That makes sense, thanks. I dropped the example because
`VariantArrayBuilder` already has some.
--
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]
