Use SlidingTimeWindowArray instead of using resetting ExpDecaySample for latency histograms

joshuacolvin0 · joshuacolvin0 · commit 00e202f69397 · 2024-01-24T11:23:10.000-07:00
diff --git a/metrics/sample.go b/metrics/sample.go
@@ -33,9 +33,7 @@ type Sample interface {
 }
 
 func NewBoundedHistogramSample() Sample {
-	return ResettingSample(
-		NewExpDecaySample(1028, 0.015),
-	)
+	return NewSlidingTimeWindowArraySample(time.Minute * 1)
 }
 
 // ExpDecaySample is an exponentially-decaying sample using a forward-decaying

Original file line number	Diff line number	Diff line change
`@@ -33,9 +33,7 @@ type Sample interface {`
`33`	`33`	`}`
`34`	`34`
`35`	`35`	`func NewBoundedHistogramSample() Sample {`
`36`		`- return ResettingSample(`
`37`		`- NewExpDecaySample(1028, 0.015),`
`38`		`- )`
	`36`	`+ return NewSlidingTimeWindowArraySample(time.Minute * 1)`
`39`	`37`	`}`
`40`	`38`
`41`	`39`	`// ExpDecaySample is an exponentially-decaying sample using a forward-decaying`