[aggregator] Cap length of sample for reusal (m3db#3536)

yyin-sc · Jun 3, 2021 · feba416 · feba416
1 parent e54033f
commit feba416
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/src/aggregator/aggregation/quantile/cm/list.go b/src/aggregator/aggregation/quantile/cm/list.go
@@ -20,6 +20,8 @@
 
 package cm
 
+const _maxSamplesToReuse = 2048
+
 var (
  emptySampleList sampleList
 )
@@ -49,8 +51,13 @@ func (l *sampleList) Reset() {
  for i := range l.samples {
  l.samples[i].next, l.samples[i].prev = nil, nil
  }
+
  l.samples = l.samples[:0]
  l.free = l.free[:0]
+ if cap(l.samples) > _maxSamplesToReuse || cap(l.free) > _maxSamplesToReuse {
+ l.samples = nil
+ l.free = nil
+ }
  l.head, l.tail = nil, nil
 }