summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBrion Vibber <brion@pobox.com>2010-09-20 13:19:58 -0700
committerBrion Vibber <brion@pobox.com>2010-09-20 13:19:58 -0700
commit2f38c9c99cb10be8fab28a624fa6f64cec6c0a9b (patch)
tree341edca0a2886249c5c29553b445d62f71c4951c
parent10f6e14134cd00d018127d99f90ada5c52805e39 (diff)
Fix bug in db queue manager: when receiving an invalid or deleted notice, we were running it through the 'temporary fail, release it to try again' path instead of the 'done, discard item' path. Should fix some infinite-loop-of-doom-in-queue cases.
-rw-r--r--lib/dbqueuemanager.php2
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/dbqueuemanager.php b/lib/dbqueuemanager.php
index 3dda9fd1a..51553b899 100644
--- a/lib/dbqueuemanager.php
+++ b/lib/dbqueuemanager.php
@@ -100,7 +100,7 @@ class DBQueueManager extends QueueManager
}
} else {
$this->_log(LOG_INFO, "[$queue] Got empty/deleted item, discarding");
- $this->_fail($qi);
+ $this->_done($qi);
}
return true;
}