From 1e2d6875a08364d989c8ecb0afdb932ba844322c Mon Sep 17 00:00:00 2001 From: Edd Robinson Date: Fri, 18 Dec 2020 15:40:26 +0000 Subject: [PATCH] refactor: update segment_store/src/segment.rs Co-authored-by: Andrew Lamb --- segment_store/src/segment.rs | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/segment_store/src/segment.rs b/segment_store/src/segment.rs index 805df731bb..35a19665db 100644 --- a/segment_store/src/segment.rs +++ b/segment_store/src/segment.rs @@ -782,10 +782,11 @@ impl Segment { } } None => { - let mut group_key_aggs = Vec::with_capacity(dst.aggregate_columns.len()); - for (_, agg_type) in &dst.aggregate_columns { - group_key_aggs.push(AggregateResult::from(agg_type)); - } + let mut group_key_aggs = dst + .aggregate_columns + .iter() + .map(|(_, agg_type)| AggregateResult::from(agg_type)) + .collect::>(); for (i, values) in aggregate_columns_data.iter().enumerate() { group_key_aggs[i].update(values.value(row));