comments

Author: friendlymatthew

parquet-variant/benches/variant_builder.rs

@@ -496,9 +496,9 @@ fn bench_iteration_performance(c: &mut Criterion) {
 }
 
 fn bench_extend_metadata_builder(c: &mut Criterion) {
-    let list = (0..u32::MAX).map(|i| format!("id_{i}")).collect::<Vec<_>>();
+    let list = (0..400_000).map(|i| format!("id_{i}")).collect::<Vec<_>>();
 
-    c.bench_function("bench_validate_large_nested_list", |b| {
+    c.bench_function("bench_extend_metadata_builder", |b| {
         b.iter(|| {
             std::hint::black_box(
                 VariantBuilder::new().with_field_names(list.iter().map(|s| s.as_str())),

parquet-variant/src/builder.rs

@@ -403,9 +403,9 @@ impl<S: AsRef<str>> FromIterator<S> for MetadataBuilder {
 impl<S: AsRef<str>> Extend<S> for MetadataBuilder {
     fn extend<T: IntoIterator<Item = S>>(&mut self, iter: T) {
         let iter = iter.into_iter();
-        let (min, max) = iter.size_hint();
+        let (min, _) = iter.size_hint();
 
-        self.field_names.reserve(max.unwrap_or(min));
+        self.field_names.reserve(min);
 
         for field_name in iter {
             self.upsert_field_name(field_name.as_ref());