Removing metrics for queues #88

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants

gphat commented Feb 23, 2012

This patch doesn't seem to work until a newer ostrich is included with Kestrel, but there are a lot of queue metrics hanging around post-deletion in 2.1.5.

I had trouble testing this because I can't seem to get the latest version of Ostrich into Kestrel because one is using 2.9.1 and the other 2.8.1.

I'll just leave this here until it can be applied?

gphat commented Feb 27, 2012

I know you've already got a pile of fixes and stuff to release, but I believe that this problem is causing our ostrich thread to OOM. We use a lot of transient queues. :(

gphat commented Feb 27, 2012

I left out create_time from being purged, so this fixes that. Also, i noticed that the ostrich release before the current one works fine as a dep, so I included that. I'm running this version in our infrastructure due to the leaks associated oom'ing things. :(

Contributor

robey commented Mar 21, 2012

sorry, i totally lost track of this one.

Contributor

robey commented Mar 21, 2012

merged!

robey closed this Mar 21, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment