@@ -220,19 +220,19 @@ impl VariantBuilderExt for VariantArrayBuilder {
220220/// # Example:
221221/// ```
222222/// # use arrow::array::Array;
223- /// # use parquet_variant::{Variant, EMPTY_VARIANT_METADATA };
223+ /// # use parquet_variant::{Variant};
224224/// # use parquet_variant_compute::VariantValueArrayBuilder;
225225/// // Create a variant value builder for 10 rows
226226/// let mut builder = VariantValueArrayBuilder::new(10);
227227///
228228/// // Append some values with their corresponding metadata
229- /// // In practice, you should use the existing metadata you have access to .
230- /// builder.append_value(Variant::from(42), EMPTY_VARIANT_METADATA).unwrap( );
229+ /// // In practice, some of the variant values would be objects with internal metadata .
230+ /// builder.append_value(Variant::from(42));
231231/// builder.append_null();
232- /// builder.append_value(Variant::from("hello"), EMPTY_VARIANT_METADATA).unwrap( );
232+ /// builder.append_value(Variant::from("hello"));
233233///
234234/// // Build the final value array
235- /// let value_array = builder.build();
235+ /// let value_array = builder.build().unwrap() ;
236236/// assert_eq!(value_array.len(), 3);
237237/// ```
238238#[ derive( Debug ) ]
@@ -289,10 +289,10 @@ impl VariantValueArrayBuilder {
289289 ///
290290 /// # Example
291291 /// ```
292- /// # use parquet_variant::{ Variant, EMPTY_VARIANT_METADATA} ;
292+ /// # use parquet_variant::Variant;
293293 /// # use parquet_variant_compute::VariantValueArrayBuilder;
294294 /// let mut builder = VariantValueArrayBuilder::new(10);
295- /// builder.append_value(Variant::from(42), EMPTY_VARIANT_METADATA).unwrap( );
295+ /// builder.append_value(Variant::from(42));
296296 /// ```
297297 pub fn append_value ( & mut self , value : Variant < ' _ , ' _ > ) {
298298 let metadata = value. metadata ( ) . cloned ( ) . unwrap_or ( EMPTY_VARIANT_METADATA ) ;
0 commit comments