Skip to content

Conversation

charleskorn
Copy link
Contributor

What this PR does

This PR fixes an issue where a SeriesMetadata slice is used after being returned to the pool. This could lead to unexpected behaviour, including incorrect query results and panics.

Which issue(s) this PR fixes or relates to

(none)

Checklist

  • [n/a] Tests updated.
  • [n/a] Documentation added.
  • [covered by Mimir Query Engine #10067] CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX].
  • [n/a] about-versioning.md updated with experimental features.

@charleskorn charleskorn marked this pull request as ready for review May 7, 2025 00:10
@charleskorn charleskorn requested a review from a team as a code owner May 7, 2025 00:10
Copy link
Contributor

@56quarters 56quarters left a comment

Choose a reason for hiding this comment

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

LGTM, nice catch.

@charleskorn charleskorn merged commit 7d8d50c into main May 7, 2025
30 checks passed
@charleskorn charleskorn deleted the charleskorn/fix-race branch May 7, 2025 00:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants