With the current logic, waiting queue will never actually time its workers out per the processing time limit; this is because the inner while() loop will just keep on repeating. Attached patch fixes it. It now checks after a claim has returned an job to ensure that we're still within the time limit. If we are, it processes the job; if not, it releases the job back to the queue, watchdogs and breaks out of the outer while.

CommentFileSizeAuthor
wq-respect-time.patch1.49 KBsdboyer

Comments

Anonymous’s picture

Status: Needs review » Fixed

blurgh, thanks for fixing the brown-paper-bag release.

http://drupal.org/cvs?commit=490892

Status: Fixed » Closed (fixed)
Issue tags: -git phase 2, -git sprint 10

Automatically closed -- issue fixed for 2 weeks with no activity.