search

gresrun / jesque

An implementation of Resque in Java.
http://gresrun.github.io/jesque
Apache License 2.0
628 stars 131 forks source link

proper resizing of failed item queue #167

Closed bp-FLN closed 2 years ago

bp-FLN commented 4 years ago

the number of items to keep in the failed queue can not be downsized. the problem is that we always only remove max. 1 item with the ltrim. the fix is to trim the list to exactly the size we want. we also don't need to check the size of the failed queue before trimming, as we can simply ltrim all the time now.

coveralls commented 4 years ago

Coverage Status

Coverage increased (+0.2%) to 71.024% when pulling 48db99f0b39a07d13f4c7c4c171f75d4ba6dfdd3 on uberall:fail-queue-max-items-resize into b55a7c821534faf0083881974e566bc4bc588e11 on gresrun:master.