Skip to content

Conversation

@Hilbrand
Copy link
Member

When RabbitMQ runs out of memory it will temporary block tasks on the queue. The taskmanager will than get disconnected. It will handle this, but it didn't update this correctly in the load statistics. As a result the taskmanager reported it was busy while it wasn't actually. This will result in few to none new tasks being put on the queue.

The fix here is a change in the QueueWatchDog. That class now does the actually watching and calls reset to all relevant processes when a watch dog situation appears. Added unit test to test reset conditions in the different classes.

Also update to use the latest aerius parent root pom (to also get ride of junit startup issues in Eclipse).

…trigger

When RabbitMQ runs out of memory it will temporary block tasks on the queue. The taskmanager will than get disconnected.
It will handle this, but it didn't update this correctly in the load statistics. As a result the taskmanager reported it was busy while it wasn't actually.
This will result in few to none new tasks being put on the queue.

The fix here is a change in the QueueWatchDog. That class now does the actually watching and calls reset to all relevant processes when a watch dog situation appears.
Added unit test to test reset conditions in the different classes.

Also update to use the latest aerius parent root pom (to also get ride of junit startup issues in Eclipse).
Copy link
Member

@BertScholten BertScholten left a comment

Choose a reason for hiding this comment

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

LGTM

@SerhatG
Copy link
Member

SerhatG commented Jan 14, 2026

Would it be also helpful to detect blocked connections and try to handle them gracefully? ( https://www.rabbitmq.com/docs/connection-blocked )
If done properly the connections shouldn't drop. These changes are still useful of course, to have as fallback in case of any (new) issues.

@Hilbrand Hilbrand requested a review from BertScholten January 14, 2026 16:17
Copy link
Member

@BertScholten BertScholten left a comment

Choose a reason for hiding this comment

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

LGTM

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