Skip to content

Block on kombu producer pool acquire instead of raising LimitExceeded#461

Merged
mvdbeek merged 1 commit into
galaxyproject:masterfrom
mvdbeek:fix-amqp-producer-pool-limit
Jun 11, 2026
Merged

Block on kombu producer pool acquire instead of raising LimitExceeded#461
mvdbeek merged 1 commit into
galaxyproject:masterfrom
mvdbeek:fix-amqp-producer-pool-limit

Conversation

@mvdbeek

@mvdbeek mvdbeek commented Jun 6, 2026

Copy link
Copy Markdown
Member

When many jobs complete simultaneously, all postprocess threads compete for the 10-slot kombu producer pool. The non-blocking acquire() raises LimitExceeded which is not in recoverable_exceptions, crashing the thread and losing the status update. Blocking waits for a free slot instead.

Reported by @martindemko

When many jobs complete simultaneously, all postprocess threads compete
for the 10-slot kombu producer pool. The non-blocking acquire() raises
LimitExceeded which is not in recoverable_exceptions, crashing the thread
and losing the status update. Blocking waits for a free slot instead.
@martindemko

Copy link
Copy Markdown

Thank you, you are a savior. It was a batch of thousands of small jobs (a hundred of them running in parallel), so it was very possible that a few of them finished at the sme time.

@mvdbeek mvdbeek merged commit 088d185 into galaxyproject:master Jun 11, 2026
21 checks passed
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