Yelp / doloop

Task loop for keeping things updated
Other
10 stars 10 forks source link

End to stats() starvation on high-load systems. #24

Closed yarrrtem closed 12 years ago

yarrrtem commented 12 years ago

Ever since stats() was converted to a read-only connection, on systems running close to capacity the write locks from get(), did(), and bump() were starving stats() calls. Moving the WRITE locks to LOW_PRIORITY WRITE locks should alleviate this issue.

coyotemarin commented 12 years ago

We decided not to do this; better to starve stats() than actual doloop workers.