worker_roulette 0.1.9 → 0.1.12

checksums.yaml

@@ -1,15 +1,7 @@
 ---
-!binary "U0hBMQ==":
-  metadata.gz: !binary |-
-    MzJmOTEyZjdhMWMyYmIzODBiOWFmZTdmNDE5NGUzM2U4MDdlMGYxYg==
-  data.tar.gz: !binary |-
-    MDNhZWRmYjVkOWQ1MGEwYjk3NmQ5Njg5YjViMzIwZWFkYzU5NjMxMA==
+SHA1:
+  metadata.gz: 18fac5a135506a2e24d37a0d7c64a5e38deb5958
+  data.tar.gz: 8805d12a5a60b90353baf73a459399e41c0a7254
 SHA512:
-  metadata.gz: !binary |-
-    MGNhMjYxYTkxY2UwZGJmMDZjNTc1MzY0N2Y4MTYzMWJkZTE5N2RiYjZkNjIz
-    MzQ4NmVlYTU5ZjU2MGZmNWVhYTg5NDRiNWRlZTBlNzMxZDk2ZWE4OTE1OTg2
-    Y2Q1OTBhMmVhN2IwNmQ3OGQ3ZDA1MTI4MDdmMzA1MWJkZTZkYzc=
-  data.tar.gz: !binary |-
-    Y2I0ZGFkOTI2OGNjODA5M2NmMTQxM2Q2MTNhN2IzMTVlZDZkNWI1ZjU2Nzll
-    OTUxNjhkOTFjYzU4NzQ3OTU5OTMxYjI3OGZmY2NiMzlhNmM0NTMwYTg5ZGZj
-    ZGE5MjliZWI0MTE2MjAzMWM5MTY3ODhmMzhlYjYyMmNhZTdjYWM=
+  metadata.gz: a924c28f238c090ceaaab24ce7c0d1c247711f0e253c1ab63ede4b8515fc7eddf2d1139aba6c8abf570d2c0270a1bba3c772dd3eee9c78826f63197b65220eb5
+  data.tar.gz: 9e8dc7891d83aad2e8ed176a25588cd0fd32ee853e7b7819cb21afdf8eab311184ccd52c5133d36e72e87ec6fe323c25051e704e6ab291e375b1df516de8f688

data/.agignore

@@ -0,0 +1,2 @@
+coverage/*
+vendor/bundle/*

data/lib/worker_roulette/foreman.rb

@@ -3,33 +3,33 @@ module WorkerRoulette
     attr_reader :sender, :namespace, :channel
 
     LUA_ENQUEUE_WORK_ORDERS = <<-HERE
-        local counter_key       = KEYS[1]
-        local job_board_key     = KEYS[2]
-        local sender_key        = KEYS[3]
-        local channel           = KEYS[4]
+      local counter_key       = KEYS[1]
+      local job_board_key     = KEYS[2]
+      local sender_key        = KEYS[3]
+      local channel           = KEYS[4]
 
-        local work_order        = ARGV[1]
-        local job_notification  = ARGV[2]
-        local redis_call        = redis.call
-        local zscore            = 'ZSCORE'
-        local incr              = 'INCR'
-        local zadd              = 'ZADD'
-        local rpush             = 'RPUSH'
-        local publish           = 'PUBLISH'
-        local zcard             = 'ZCARD'
-        local del               = 'DEL'
+      local work_order        = ARGV[1]
+      local job_notification  = ARGV[2]
+      local redis_call        = redis.call
+      local zscore            = 'ZSCORE'
+      local incr              = 'INCR'
+      local zadd              = 'ZADD'
+      local rpush             = 'RPUSH'
+      local publish           = 'PUBLISH'
+      local zcard             = 'ZCARD'
+      local del               = 'DEL'
 
-        local function enqueue_work_orders(work_order, job_notification)
-          redis_call(rpush, sender_key, work_order)
+      local function enqueue_work_orders(work_order, job_notification)
+        redis_call(rpush, sender_key, work_order)
 
-          -- called when a work from a new sender is added
-          if (redis_call(zscore, job_board_key, sender_key) == false) then
-            local count     = redis_call(incr, counter_key)
-            redis_call(zadd, job_board_key, count, sender_key)
-          end
+        -- called when a work from a new sender is added
+        if (redis_call(zscore, job_board_key, sender_key) == false) then
+          local count     = redis_call(incr, counter_key)
+          redis_call(zadd, job_board_key, count, sender_key)
         end
+      end
 
-        enqueue_work_orders(work_order, job_notification)
+      enqueue_work_orders(work_order, job_notification)
     HERE
 
     def initialize(redis_pool, sender, namespace = nil)
@@ -47,7 +47,7 @@ module WorkerRoulette
 
     def enqueue_work_order_without_headers(work_order, &callback)
       @lua.call(LUA_ENQUEUE_WORK_ORDERS, [counter_key, job_board_key, sender_key, @channel],
-               [WorkerRoulette.dump(work_order),  WorkerRoulette::JOB_NOTIFICATIONS], &callback)
+                [WorkerRoulette.dump(work_order),  WorkerRoulette::JOB_NOTIFICATIONS], &callback)
     end
 
     def job_board_key
@@ -68,4 +68,4 @@ module WorkerRoulette
       Hash['sender' => sender]
     end
   end
-end
+end

data/lib/worker_roulette/lua.rb

@@ -8,7 +8,7 @@ module WorkerRoulette
 
     def call(lua_script, keys_accessed = [], args = [], &callback)
       @connection_pool.with do |redis|
-        results = evalsha(redis, lua_script, keys_accessed, args, &callback)
+        evalsha(redis, lua_script, keys_accessed, args, &callback)
       end
     end
 
@@ -26,14 +26,14 @@ module WorkerRoulette
 
     def eval(redis, lua_script, keys_accessed, args, &callback)
       results = redis.eval(lua_script, keys_accessed.length, *keys_accessed, *args)
-      results.callback &callback if callback
+      results.callback(&callback) if callback
       results.errback  {|err_msg| raise EM::Hiredis::RedisError.new(err_msg)}
     end
 
     def evalsha(redis, lua_script, keys_accessed, args, &callback)
       if redis.class == EM::Hiredis::Client
         results = redis.evalsha(sha(lua_script), keys_accessed.length, *keys_accessed, *args)
-        results.callback &callback if callback
+        results.callback(&callback) if callback
         results.errback {eval(redis, lua_script, keys_accessed, args, &callback)}
       else
         begin
@@ -47,4 +47,4 @@ module WorkerRoulette
       results
     end
   end
-end
+end

data/lib/worker_roulette/tradesman.rb

@@ -1,71 +1,106 @@
-require 'timers'
-
 module WorkerRoulette
   class Tradesman
     attr_reader :last_sender, :remaining_jobs, :timer
 
     LUA_DRAIN_WORK_ORDERS = <<-HERE
-    local empty_string      = ""
-    local job_board_key     = KEYS[1]
-    local last_sender_key   = KEYS[2] or empty_string
-    local sender_key        = ARGV[1] or empty_string
-    local redis_call        = redis.call
-    local lock_key_prefix   = "L*:"
-    local lock_value        = 1
-    local ex                = "EX"
-    local nx                = "NX"
-    local get               = "GET"
-    local set               = "SET"
-    local del               = "DEL"
-    local lrange            = "LRANGE"
-    local zrank             = "ZRANK"
-    local zrange            = "ZRANGE"
-    local zrem              = "ZREM"
-    local zcard             = 'ZCARD'
-
-    local function drain_work_orders(job_board_key, last_sender_key, sender_key)
-
-    --kill lock for last_sender_key
-    if last_sender_key ~= empty_string then
-      local last_sender_lock_key = lock_key_prefix .. last_sender_key
-      redis_call(del, last_sender_lock_key)
-    end
+      local empty_string      = ""
+      local job_board_key     = KEYS[1]
+      local last_sender_key   = KEYS[2] or empty_string
+      local sender_key        = ARGV[1] or empty_string
+      local redis_call        = redis.call
+      local lock_key_prefix   = "L*:"
+      local lock_value        = 1
+      local ex                = "EX"
+      local nx                = "NX"
+      local get               = "GET"
+      local set               = "SET"
+      local del               = "DEL"
+      local lrange            = "LRANGE"
+      local zrank             = "ZRANK"
+      local zrange            = "ZRANGE"
+      local zrem              = "ZREM"
+      local zcard             = 'ZCARD'
+
+      local function drain_work_orders(job_board_key, last_sender_key, sender_key)
+
+        --kill lock for last_sender_key
+        if last_sender_key ~= empty_string then
+          local last_sender_lock_key = lock_key_prefix .. last_sender_key
+          redis_call(del, last_sender_lock_key)
+        end
+
+        if sender_key == empty_string then
+          sender_key = redis_call(zrange, job_board_key, 0, 0)[1] or empty_string
+
+          -- return if job_board is empty
+          if sender_key == empty_string then
+            return {empty_string, {}, 0}
+          end
+        end
+
+        local lock_key       = lock_key_prefix .. sender_key
+        local was_not_locked = redis_call(set, lock_key, lock_value, ex, 3, nx)
+
+        if was_not_locked then
+          local work_orders    = redis_call(lrange, sender_key, 0, -1)
+          redis_call(del, sender_key)
 
-    if sender_key == empty_string then
-      sender_key = redis_call(zrange, job_board_key, 0, 0)[1] or empty_string
+          redis_call(zrem, job_board_key, sender_key)
+          local remaining_jobs = redis_call(zcard, job_board_key) or 0
 
-      -- return if job_board is empty
-      if sender_key == empty_string then
-        return {empty_string, {}, 0}
+          return {sender_key, work_orders, remaining_jobs}
+        else
+          local sender_index    = redis_call(zrank, job_board_key, sender_key)
+          local next_index      = sender_index + 1
+          local next_sender_key = redis_call(zrange, job_board_key, next_index, next_index)[1]
+          if next_sender_key then
+            return drain_work_orders(job_board_key, empty_string, next_sender_key)
+          else
+            -- return if job_board is empty
+            return {empty_string, {}, 0}
+          end
+        end
       end
-    end
 
-    local lock_key       = lock_key_prefix .. sender_key
-    local was_not_locked = redis_call(set, lock_key, lock_value, ex, 3, nx)
+      return drain_work_orders(job_board_key, last_sender_key, empty_string)
+    HERE
+
+    LUA_DRAIN_WORK_ORDERS_FOR_SENDER = <<-THERE
+      local empty_string     = ""
+      local job_board_key    = KEYS[1]
+      local sender_key       = KEYS[2] or empty_string
+      local redis_call       = redis.call
+      local lock_key_prefix  = "L*:"
+      local lock_value       = 1
+      local get              = "GET"
+      local del              = "DEL"
+      local lrange           = "LRANGE"
+      local zrange           = "ZRANGE"
+      local zrem             = "ZREM"
+
+      local function drain_work_orders_for_sender(job_board_key, sender_key)
+
+        if sender_key == empty_string then
+          return {empty_string, {}, 0}
+        end
 
-    if was_not_locked then
-      local work_orders    = redis_call(lrange, sender_key, 0, -1)
-      redis_call(del, sender_key)
+        local lock_key = lock_key_prefix .. sender_key
+        was_locked = redis_call(get, lock_key)
 
-      redis_call(zrem, job_board_key, sender_key)
-      local remaining_jobs = redis_call(zcard, job_board_key) or 0
+        if was_locked == 1 then
+          local work_orders = redis_call(lrange, sender_key, 0, -1)
+          redis_call(del, sender_key)
 
-      return {sender_key, work_orders, remaining_jobs}
-    else
-      local sender_index    = redis_call(zrank, job_board_key, sender_key)
-      local next_index      = sender_index + 1
-      local next_sender_key = redis_call(zrange, job_board_key, next_index, next_index)[1]
-      if next_sender_key then
-        return drain_work_orders(job_board_key, empty_string, next_sender_key)
-      else
-        -- return if job_board is empty
-        return {empty_string, {}, 0}
+          redis_call(zrem, job_board_key, sender_key)
+
+          return { sender_key, work_orders }
+        else
+          return { sender_key, {} }
+        end
       end
-    end
-  end
 
-  return drain_work_orders(job_board_key, last_sender_key, empty_string)
-  HERE
+      return drain_work_orders_for_sender(job_board_key, sender_key)
+    THERE
 
     def initialize(redis_pool, evented, namespace = nil, polling_time = WorkerRoulette::DEFAULT_POLLING_TIME)
       @evented        = evented
@@ -73,7 +108,6 @@ module WorkerRoulette
       @redis_pool     = redis_pool
       @namespace      = namespace
       @channel        = namespace || WorkerRoulette::JOB_NOTIFICATIONS
-      @timer          = Timers::Group.new
       @lua            = Lua.new(@redis_pool)
       @remaining_jobs = 0
     end
@@ -102,16 +136,40 @@ module WorkerRoulette
       end
     end
 
+    def get_more_work_for_last_sender(&on_message_callback)
+      return unless on_message_callback
+      more_work_orders! do |work|
+        on_message_callback.call(work) if work.any?
+        if @evented
+          evented_drain_work_queue!(&on_message_callback)
+        else
+          non_evented_drain_work_queue!(&on_message_callback)
+        end
+      end
+    end
+
+    def more_work_orders!(&callback)
+      @lua.call(LUA_DRAIN_WORK_ORDERS_FOR_SENDER, [job_board_key, @last_sender]) do |results|
+        sender_key      = results[0]
+        raise "wrong sender key returned from LUA_DRAIN_WORK_ORDERS_FOR_SENDER" unless sender_key == @last_sender
+        work_orders     = results[1]
+        work            = work_orders.map {|work_order| WorkerRoulette.load(work_order)}
+        callback.call work if callback
+        work
+      end
+    end
+
     def job_board_key
       @job_board_key ||= WorkerRoulette.job_board_key(@namespace)
     end
 
-  private
+    private
+
     def evented_drain_work_queue!(&on_message_callback)
       if remaining_jobs > 0
         EM.next_tick {wait_for_work_orders(&on_message_callback)}
       else
-        EM.add_timer(@polling_time) {wait_for_work_orders(&on_message_callback)}
+        EM.add_timer(@polling_time) { wait_for_work_orders(&on_message_callback) }
       end
     end
 
@@ -119,7 +177,8 @@ module WorkerRoulette
       if remaining_jobs > 0
         wait_for_work_orders(&on_message_callback)
       else
-        @timer.after(@polling_time) {wait_for_work_orders(&on_message_callback)}
+        sleep 2
+        wait_for_work_orders(&on_message_callback)
       end
     end
   end

data/lib/worker_roulette/version.rb

@@ -1,3 +1,3 @@
 module WorkerRoulette
-  VERSION = "0.1.9"
+  VERSION = '0.1.12'
 end

data/spec/integration/evented_worker_roulette_spec.rb

@@ -99,6 +99,29 @@ module WorkerRoulette
         end
       end
 
+      it "should remove the lock from the last_sender's queue" do
+        most_recent_sender = 'most_recent_sender'
+        most_recent_foreman = worker_roulette.foreman(most_recent_sender)
+        other_foreman = worker_roulette.foreman('katie_80')
+
+        other_foreman.enqueue_work_order(work_orders) do
+          most_recent_foreman.enqueue_work_order(work_orders) do
+            expect(redis.keys("L*:*").length).to eq(0)
+            subject.work_orders! do
+              expect(redis.get("L*:katie_80")).to eq("1")
+              expect(redis.keys("L*:*").length).to eq(1)
+              subject.work_orders! do
+                expect(redis.keys("L*:*").length).to eq(1)
+                expect(redis.get("L*:most_recent_sender")).to eq("1")
+                subject.work_orders!
+                done(0.2) do
+                  expect(redis.keys("L*:*").length).to eq(0)
+                end
+              end
+            end
+          end
+        end
+      end
 
       it "should drain one set of work_orders from the sender's slot in the job board" do
         foreman.enqueue_work_order(work_orders) do

data/spec/integration/worker_roulette_spec.rb

@@ -1,5 +1,9 @@
 require "spec_helper"
 module WorkerRoulette
+  class Tradesman
+    attr_reader :lua
+  end
+
   describe WorkerRoulette do
     let(:sender) {'katie_80'}
     let(:work_orders) {["hello", "foreman"]}
@@ -12,6 +16,10 @@ module WorkerRoulette
 
     let(:redis) {Redis.new(worker_roulette.redis_config)}
 
+    before do
+      redis.flushall
+    end
+
     it "should exist" do
       expect(worker_roulette).to be_instance_of(WorkerRoulette)
     end
@@ -65,35 +73,52 @@ module WorkerRoulette
     end
 
     context Tradesman do
-      let(:foreman) {worker_roulette.foreman(sender)}
-      let(:subject)  {worker_roulette.tradesman}
+      let(:foreman)        { worker_roulette.foreman(sender) }
+      subject(:tradesman)  { worker_roulette.tradesman }
 
       before do
         foreman.enqueue_work_order(work_orders)
       end
 
+      context 'removing locks from queues' do
+        it "for the last_sender's queue" do
+          most_recent_sender = 'most_recent_sender'
+          most_recent_foreman = worker_roulette.foreman(most_recent_sender)
+          most_recent_foreman.enqueue_work_order(work_orders)
+          expect(redis.keys("L*:*").length).to eq(0)
+          tradesman.work_orders!
+          expect(redis.get("L*:katie_80")).to eq("1")
+          expect(redis.keys("L*:*").length).to eq(1)
+          tradesman.work_orders!
+          expect(redis.keys("L*:*").length).to eq(1)
+          expect(redis.get("L*:most_recent_sender")).to eq("1")
+          tradesman.work_orders!
+          expect(redis.keys("L*:*").length).to eq(0)
+        end
+      end
+
       it "should have a last sender if it found messages" do
-        expect(subject.work_orders!.length).to eq(1)
-        expect(subject.last_sender).to eq(sender)
+        expect(tradesman.work_orders!.length).to eq(1)
+        expect(tradesman.last_sender).to eq(sender)
       end
 
       it "should not have a last sender if it found no messages" do
-        expect(subject.work_orders!.length).to eq(1)
-        expect(subject.work_orders!.length).to eq(0)
-        expect(subject.last_sender).to be_nil
+        expect(tradesman.work_orders!.length).to eq(1)
+        expect(tradesman.work_orders!.length).to eq(0)
+        expect(tradesman.last_sender).to be_nil
       end
 
       it "should drain one set of work_orders from the sender's work queue" do
-        expect(subject.work_orders!).to eq([work_orders_with_headers])
-        expect(subject.work_orders!).to be_empty
-        expect(subject.work_orders!).to be_empty #does not throw an error if queue is already empty
+        expect(tradesman.work_orders!).to eq([work_orders_with_headers])
+        expect(tradesman.work_orders!).to be_empty
+        expect(tradesman.work_orders!).to be_empty #does not throw an error if queue is already empty
       end
 
       it "should drain all the work_orders from the sender's work queue" do
         foreman.enqueue_work_order(work_orders)
-        expect(subject.work_orders!).to eq([work_orders_with_headers, work_orders_with_headers])
-        expect(subject.work_orders!).to be_empty
-        expect(subject.work_orders!).to be_empty #does not throw an error if queue is already empty
+        expect(tradesman.work_orders!).to eq([work_orders_with_headers, work_orders_with_headers])
+        expect(tradesman.work_orders!).to be_empty
+        expect(tradesman.work_orders!).to be_empty #does not throw an error if queue is already empty
       end
 
       it "should take the oldest sender off the job board (FIFO)" do
@@ -101,21 +126,16 @@ module WorkerRoulette
         most_recent_sender = 'most_recent_sender'
         most_recent_foreman = worker_roulette.foreman(most_recent_sender)
         most_recent_foreman.enqueue_work_order(work_orders)
-        expect(redis.zrange(subject.job_board_key, 0, -1)).to eq([oldest_sender, most_recent_sender])
-        subject.work_orders!
-        expect(redis.zrange(subject.job_board_key, 0, -1)).to eq([most_recent_sender])
+        expect(redis.zrange(tradesman.job_board_key, 0, -1)).to eq([oldest_sender, most_recent_sender])
+        tradesman.work_orders!
+        expect(redis.zrange(tradesman.job_board_key, 0, -1)).to eq([most_recent_sender])
       end
 
       it "should get the work_orders from the next queue when a new job is ready, then poll for new work" do
-        subject.work_orders!
-        expect(subject).to receive(:work_orders!).and_call_original
-        expect(subject.timer).to receive(:after)
-
-        foreman.enqueue_work_order(work_orders)
-
-        subject.wait_for_work_orders do |redis_work_orders|
+        tradesman.wait_for_work_orders do |redis_work_orders|
           expect(redis_work_orders).to eq([work_orders_with_headers])
-          expect(subject.last_sender).to eq('katie_80')
+          expect(tradesman.last_sender).to eq('katie_80')
+          allow(tradesman).to receive(:wait_for_work_orders)
         end
       end
 
@@ -133,7 +153,28 @@ module WorkerRoulette
           expect(work.to_s).to match("some old fashion work")
           expect(work.to_s).not_to match("evil")
           expect(tradesman.last_sender).to eq('foreman')
+          allow(tradesman).to receive(:wait_for_work_orders)
+        end
+      end
+
+      it "goes back to the channel to get more work for the same sender" do
+        tradesman.wait_for_work_orders do |redis_work_orders|
+          expect(redis_work_orders).to eq([work_orders_with_headers])
+          expect(tradesman.last_sender).to eq('katie_80')
+          allow(tradesman).to receive(:wait_for_work_orders)
         end
+
+        expect(tradesman.lua).to receive(:call).with(Tradesman::LUA_DRAIN_WORK_ORDERS_FOR_SENDER, [instance_of(String), sender])
+        tradesman.get_more_work_for_last_sender do |redis_work_orders|
+          expect(redis_work_orders).to eq([])
+        end
+
+        foreman.enqueue_work_order("more_work_orders")
+        expect(tradesman.lua).to receive(:call).with(Tradesman::LUA_DRAIN_WORK_ORDERS_FOR_SENDER, [instance_of(String), sender])
+        tradesman.get_more_work_for_last_sender do |redis_work_orders|
+          expect(redis_work_orders).to eq(["more_work_orders"])
+        end
+
       end
 
       context "Failure" do
@@ -149,15 +190,6 @@ module WorkerRoulette
             expect(pooled_redis.info["connected_clients"].to_i).to be > (worker_roulette.pool_size)
           end
         end
-
-        #This may be fixed soon (10 Feb 2014 - https://github.com/redis/redis-rb/pull/389 and https://github.com/redis/redis-rb/issues/364)
-        it "should not be fork() proof -- forking reconnects need to be handled in the calling code (until redis gem is udpated, then we should be fork-proof)" do
-          instance = WorkerRoulette.start
-          instance.tradesman_connection_pool.with {|pooled_redis| pooled_redis.get("foo")}
-          fork do
-            expect {instance.tradesman_connection_pool.with {|pooled_redis| pooled_redis.get("foo")}}.to raise_error(Redis::InheritedError)
-          end
-        end
       end
     end
   end

data/worker_roulette.gemspec

@@ -10,28 +10,25 @@ Gem::Specification.new do |spec|
   spec.email         = ['classicist@gmail.com']
   spec.description   = %q{Pub Sub Queue for Redis that ensures ordered processing}
   spec.summary       = %q{Pub Sub Queue for Redis that ensures ordered processing}
-  spec.homepage      = 'https://github.com/nexiahome/worker_roulette'
+  spec.homepage      = 'https://github.com/classicist/worker_roulette'
 
   spec.files         = `git ls-files`.split($/)
   spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
   spec.test_files    = spec.files.grep(%r{^(spec)/})
   spec.require_paths = ['lib']
 
-  spec.add_dependency 'oj'
-  spec.add_dependency 'redis', '~> 3.0.7'
-  spec.add_dependency 'hiredis', '~> 0.4.5'
-  spec.add_dependency 'em-hiredis', '~> 0.2.1'
-  spec.add_dependency 'connection_pool'
-  spec.add_dependency 'eventmachine', '~> 1.0.3'
-  spec.add_dependency 'timers', '~> 3.0.1'
+  spec.add_dependency 'oj', '~> 2.10'
+  spec.add_dependency 'redis', '~> 3.1'
+  spec.add_dependency 'hiredis', '~> 0.5'
+  spec.add_dependency 'em-hiredis', '~> 0.3'
+  spec.add_dependency 'connection_pool', '~> 2.0'
+  spec.add_dependency 'eventmachine', '~> 1.0'
 
-  spec.add_development_dependency 'bundler'
-  spec.add_development_dependency 'rake'
-  spec.add_development_dependency 'rspec', '~> 3.0.0'
-  spec.add_development_dependency 'pry-debugger'
-  spec.add_development_dependency 'simplecov'
-  spec.add_development_dependency 'simplecov-rcov'
-  spec.add_development_dependency 'rspec_junit_formatter'
-  spec.add_development_dependency 'guard'
-  spec.add_development_dependency 'guard-rspec'
+  spec.add_development_dependency 'bundler', '~> 1.7'
+  spec.add_development_dependency 'rake', '~> 10.3'
+  spec.add_development_dependency 'rspec', '~> 3.1'
+  spec.add_development_dependency 'pry-byebug', '~> 2.0'
+  spec.add_development_dependency 'simplecov', '~> 0.9'
+  spec.add_development_dependency 'simplecov-rcov', '~> 0.2'
+  spec.add_development_dependency 'rspec_junit_formatter', '~> 0.2'
 end

metadata

@@ -1,239 +1,197 @@
 --- !ruby/object:Gem::Specification
 name: worker_roulette
 version: !ruby/object:Gem::Version
-  version: 0.1.9
+  version: 0.1.12
 platform: ruby
 authors:
 - Paul Saieg
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-07-15 00:00:00.000000000 Z
+date: 2014-11-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: oj
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.10'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.10'
 - !ruby/object:Gem::Dependency
   name: redis
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.0.7
+        version: '3.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.0.7
+        version: '3.1'
 - !ruby/object:Gem::Dependency
   name: hiredis
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.4.5
+        version: '0.5'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.4.5
+        version: '0.5'
 - !ruby/object:Gem::Dependency
   name: em-hiredis
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.2.1
+        version: '0.3'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.2.1
+        version: '0.3'
 - !ruby/object:Gem::Dependency
   name: connection_pool
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: eventmachine
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.0.3
+        version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.0.3
-- !ruby/object:Gem::Dependency
-  name: timers
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ~>
-      - !ruby/object:Gem::Version
-        version: 3.0.1
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ~>
-      - !ruby/object:Gem::Version
-        version: 3.0.1
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.7'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.7'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '10.3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '10.3'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.0.0
+        version: '3.1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.0.0
+        version: '3.1'
 - !ruby/object:Gem::Dependency
-  name: pry-debugger
+  name: pry-byebug
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.9'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.9'
 - !ruby/object:Gem::Dependency
   name: simplecov-rcov
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.2'
 - !ruby/object:Gem::Dependency
   name: rspec_junit_formatter
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: guard
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: guard-rspec
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.2'
 description: Pub Sub Queue for Redis that ensures ordered processing
 email:
 - classicist@gmail.com
@@ -241,9 +199,10 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- .rspec
-- .simplecov
+- ".agignore"
+- ".gitignore"
+- ".rspec"
+- ".simplecov"
 - Gemfile
 - Guardfile
 - LICENSE.txt
@@ -264,7 +223,7 @@ files:
 - spec/unit/lua_spec.rb
 - spec/unit/readlock_spec.rb
 - worker_roulette.gemspec
-homepage: https://github.com/nexiahome/worker_roulette
+homepage: https://github.com/classicist/worker_roulette
 licenses: []
 metadata: {}
 post_install_message: 
@@ -273,17 +232,17 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.4.3
 signing_key: 
 specification_version: 4
 summary: Pub Sub Queue for Redis that ensures ordered processing