Skip to content

Commit

Permalink
Merge pull request #1443 from openmeterio/fix/remove-explicit-buckets
Browse files Browse the repository at this point in the history
fix: remove explicit buckets
  • Loading branch information
turip authored Aug 29, 2024
2 parents 492686a + 0fb746c commit 8ace391
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions openmeter/entitlement/balanceworker/recalculate.go
Original file line number Diff line number Diff line change
Expand Up @@ -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")
)
Expand Down Expand Up @@ -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)
Expand Down

0 comments on commit 8ace391

Please sign in to comment.