Skip to content

perf(blooms): return full slice to pool after use - #20981

Merged
bboreham merged 1 commit into
grafana:mainfrom
bboreham:defer-closure2
Apr 15, 2026
Merged

perf(blooms): return full slice to pool after use#20981
bboreham merged 1 commit into
grafana:mainfrom
bboreham:defer-closure2

Conversation

@bboreham

Copy link
Copy Markdown
Contributor

What this PR does / why we need it:

This is the same change as #20969, covering the remaining parts of Loki.

I hand-checked every line matching defer.*Put, and also checked using https://github.com/egonelbre/exp/tree/main/check-deferslice.

This is the same change as grafana#20969, covering the remaining parts of Loki.

Checked using https://github.com/egonelbre/exp/tree/main/check-deferslice

Signed-off-by: Bryan Boreham <bjboreham@gmail.com>
@bboreham
bboreham requested a review from a team as a code owner February 26, 2026 14:27

@trevorwhitney trevorwhitney left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@bboreham
bboreham merged commit 9d9c7af into grafana:main Apr 15, 2026
144 of 146 checks passed
@bboreham
bboreham deleted the defer-closure2 branch April 15, 2026 09:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2 participants