* @param mixed $transports name of a single queue or array of queues to pull from
* If not specified, checks all queues in the system.
*/
- static function top($transports=null) {
+ static function top($transports=null, array $ignored_transports=array()) {
$qi = new Queue_item();
if ($transports) {
$qi->transport = $transports;
}
}
+ if (!empty($ignored_transports)) {
+ // @fixme use safer escaping
+ $list = implode("','", array_map(array($qi, 'escape'), $ignored_transports));
+ $qi->whereAdd("transport NOT IN ('$list')");
+ }
$qi->orderBy('created');
$qi->whereAdd('claimed is null');
// XXX: potential race condition
// can we force it to only update if claimed is still null
// (or old)?
- common_log(LOG_INFO, 'claiming queue item id = ' . $qi->id .
- ' for transport ' . $qi->transport);
+ common_log(LOG_INFO, 'claiming queue item id = ' . $qi->getID() . ' for transport ' . $qi->transport);
$orig = clone($qi);
$qi->claimed = common_sql_now();
$result = $qi->update($orig);
if ($result) {
- common_log(LOG_INFO, 'claim succeeded.');
+ common_log(LOG_DEBUG, 'claim succeeded.');
return $qi;
} else {
- common_log(LOG_INFO, 'claim failed.');
+ common_log(LOG_ERR, 'claim of queue item id= ' . $qi->getID() . ' for transport ' . $qi->transport . ' failed.');
}
}
$qi = null;
function releaseClaim()
{
// DB_DataObject doesn't let us save nulls right now
- $sql = sprintf("UPDATE queue_item SET claimed=NULL WHERE id=%d", $this->id);
+ $sql = sprintf("UPDATE queue_item SET claimed=NULL WHERE id=%d", $this->getID());
$this->query($sql);
$this->claimed = null;