From 0fb746c2e162abd172abfc1a974573bacea45133 Mon Sep 17 00:00:00 2001 From: Peter Turi Date: Thu, 29 Aug 2024 11:35:10 +0200 Subject: [PATCH] fix: remove explicit buckets As we have moved to an int64 histogram. --- openmeter/entitlement/balanceworker/recalculate.go | 5 ----- 1 file changed, 5 deletions(-) diff --git a/openmeter/entitlement/balanceworker/recalculate.go b/openmeter/entitlement/balanceworker/recalculate.go index 8db3092e9..1fe43eb53 100644 --- a/openmeter/entitlement/balanceworker/recalculate.go +++ b/openmeter/entitlement/balanceworker/recalculate.go @@ -33,10 +33,6 @@ const ( ) var ( - // metricRecalculationBuckets are the default buckets for the recalculation time histogram. It is geared towards - // representing smaller values. - metricRecalculationBuckets = []float64{0.001, 0.005, 0.01, 0.02, 0.05, 0.075, 0.1, 0.2, 0.5, 1, 5, 10, 30, 60} - recalculationTimeUpdateAttribute = attribute.String("operation", "update") recalculationTimeDeleteAttribute = attribute.String("operation", "delete") ) @@ -91,7 +87,6 @@ func NewRecalculator(opts RecalculatorOptions) (*Recalculator, error) { metricRecalculationTime, err := opts.MetricMeter.Int64Histogram( metricNameRecalculationTime, metric.WithDescription("Entitlement recalculation time"), - metric.WithExplicitBucketBoundaries(metricRecalculationBuckets...), ) if err != nil { return nil, fmt.Errorf("failed to create recalculation time histogram: %w", err)