Skip to content

loadSnapshot: update matcher index properly while not holding lock#4714

Merged
SuperQ merged 1 commit intoprometheus:mainfrom
ultrotter:miii
Nov 12, 2025
Merged

loadSnapshot: update matcher index properly while not holding lock#4714
SuperQ merged 1 commit intoprometheus:mainfrom
ultrotter:miii

Conversation

@ultrotter
Copy link
Contributor

Since we're building the state separately, we will do the same with the matcher index, and then update it into the Silences with the lock held.

Since we're building the state separately, we will do the same with the
matcher index, and then update it into the Silences with the lock held.

Signed-off-by: Guido Trotter <guido@hudson-trading.com>
@ajaychandrak2002-bit

This comment was marked as spam.

@ajaychandrak2002-bit

This comment was marked as spam.

Copy link
Contributor

@Spaceman1701 Spaceman1701 left a comment

Choose a reason for hiding this comment

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

LGTM

@ajaychandrak2002-bit

This comment was marked as spam.

@SuperQ SuperQ merged commit 3938f63 into prometheus:main Nov 12, 2025
7 checks passed
@SoloJacobs SoloJacobs mentioned this pull request Nov 24, 2025
@ultrotter ultrotter deleted the miii branch December 3, 2025 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants