Skip to content

Conversation

yeya24
Copy link
Contributor

@yeya24 yeya24 commented Feb 17, 2025

What this PR does:

Prune buckets at the end to remove nil element to avoid panic.

Which issue(s) this PR fixes:
Fixes #6594

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

Signed-off-by: yeya24 <[email protected]>
@SungJin1212
Copy link
Member

LGTM

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Feb 18, 2025
@yeya24 yeya24 merged commit 3289a24 into cortexproject:master Feb 18, 2025
17 checks passed
@yeya24 yeya24 deleted the fix-native-histogram-bucket-marshal-error branch February 18, 2025 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/querier lgtm This PR has been approved by a maintainer size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Querier panics for native histogram with empty bucket and protobuf codec
3 participants