Skip to content

Commit

Permalink
[processor/deltotocumulative] Fix lint failure (open-telemetry#31685)
Browse files Browse the repository at this point in the history
**Description:** <Describe what has changed.>
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->

open-telemetry#31488
was recently merged and has broken `build-and-test` for all builds due
to a lint failure. It looks like two PRs were worked on in parallel, the
aforementioned one, as well as
open-telemetry#31625.
open-telemetry#31625 renamed `exp` to `stale`, but the most recently merged PR was
referencing the original `exp` variable.

This is an unreleased component, and is simply fixing a bug, so I don't
think this should have a changelog.
  • Loading branch information
crobert-1 authored and XinRanZhAWS committed Mar 13, 2024
1 parent 1ccd1fc commit 990c7a6
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions processor/deltatocumulativeprocessor/processor.go
Original file line number Diff line number Diff line change
Expand Up @@ -57,8 +57,8 @@ func newProcessor(cfg *Config, log *zap.Logger, next consumer.Metrics) *Processo
}
if cfg.MaxStreams > 0 {
lim := streams.Limit(dps, cfg.MaxStreams)
if proc.exp != nil {
lim.Evictor = proc.exp
if proc.stale != nil {
lim.Evictor = proc.stale
}
dps = lim
}
Expand Down

0 comments on commit 990c7a6

Please sign in to comment.