Skip to content

Commit 37cdec8

Browse files
committed
Merge pull request #107 from danhunsaker/feature-queue-failure
Added enqueue failure detection
2 parents b31830c + 8d1a9a5 commit 37cdec8

File tree

2 files changed

+9
-3
lines changed

2 files changed

+9
-3
lines changed

lib/Resque.php

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,11 @@ public static function fork()
9898
public static function push($queue, $item)
9999
{
100100
self::redis()->sadd('queues', $queue);
101-
self::redis()->rpush('queue:' . $queue, json_encode($item));
101+
$length = self::redis()->rpush('queue:' . $queue, json_encode($item));
102+
if ($length < 1) {
103+
return false;
104+
}
105+
return true;
102106
}
103107

104108
/**

lib/Resque/Job.php

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,11 +58,13 @@ public static function create($queue, $class, $args = null, $monitor = false)
5858
);
5959
}
6060
$id = md5(uniqid('', true));
61-
Resque::push($queue, array(
61+
if (!Resque::push($queue, array(
6262
'class' => $class,
6363
'args' => array($args),
6464
'id' => $id,
65-
));
65+
))) {
66+
return false;
67+
}
6668

6769
if($monitor) {
6870
Resque_Job_Status::create($id);

0 commit comments

Comments
 (0)