Skip to content

refactor: make the dispatcher ingestion concurrent#4967

Merged
ultrotter merged 2 commits intoprometheus:mainfrom
ultrotter:concurrentdispatcher
Feb 11, 2026
Merged

refactor: make the dispatcher ingestion concurrent#4967
ultrotter merged 2 commits intoprometheus:mainfrom
ultrotter:concurrentdispatcher

Conversation

@ultrotter
Copy link
Copy Markdown
Contributor

Based on #4958 and #4966, we can now have multiple goroutines ingesting alerts.

Copy link
Copy Markdown
Contributor

@siavashs siavashs left a comment

Choose a reason for hiding this comment

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

LGTM.

@ultrotter ultrotter force-pushed the concurrentdispatcher branch 4 times, most recently from d9b2c4e to aa864f0 Compare February 11, 2026 15:34
Guido Trotter added 2 commits February 11, 2026 10:52
We create multiple goroutines on Run(), one for maintenance, one for
start timer, and N for ingestion.

Signed-off-by: Guido Trotter <guido@hudson-trading.com>
Signed-off-by: Guido Trotter <guido@hudson-trading.com>
@ultrotter ultrotter force-pushed the concurrentdispatcher branch from aa864f0 to 3e66608 Compare February 11, 2026 15:53
@ultrotter ultrotter merged commit b089feb into prometheus:main Feb 11, 2026
7 checks passed
@ultrotter ultrotter deleted the concurrentdispatcher branch February 11, 2026 16:12
SoloJacobs pushed a commit to SoloJacobs/alertmanager that referenced this pull request Mar 15, 2026
* [dispatcher] increase concurrency of alert ingestion

We create multiple goroutines on Run(), one for maintenance, one for
start timer, and N for ingestion.

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

* decrease concurrency to avoid some contention on channels and ag locks

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

---------

Signed-off-by: Guido Trotter <guido@hudson-trading.com>
Co-authored-by: Guido Trotter <guido@hudson-trading.com>
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.

3 participants