action_subscriber 2.1.1-java → 2.1.2-java

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0bb53b1d4649ffe66409823cf35a69ffc3c2f773
-  data.tar.gz: 807b0206c26e9033012b6872e3291db06820d26d
+  metadata.gz: 0f7cb8c3f5be3679d9626c67fe5778869efa7ec2
+  data.tar.gz: 91024f89d188ef02703468d24711c47c073eee20
 SHA512:
-  metadata.gz: 2fa2f6df1b924d98b696bea64acfbfb592dee42abfe5a17cfe2012e30ae619aeee8fce1d1d43a59d11c015f8f7153010aa5800a9219e2cb1067a47a3db6171e0
-  data.tar.gz: a92dc35e85dc2e7bceef7d01ee9b0a463659f8360a773df06556c28078e28d6ed671f91991dc4e365fe0e6c712ddfbff06af7ec6786e4c89c7d11ebdfbc2bf3a
+  metadata.gz: cc9301af401082e92f3d5e3930e0de5e1aaa4bac0d581c4646c27ea8f20f7ef38e0c963c1038677abb966035aa5996b749fde7934bfce039143de9cfe2ecb0f8
+  data.tar.gz: 160e1bac79273ea4af61819402b6f789b820ce307ab4d2011a2fedd347c00c5e76cabd98646433191597f85881aae1a36ea68c59dfd4724f8c425bd93f51e056

data/action_subscriber.gemspec

@@ -26,7 +26,7 @@ Gem::Specification.new do |spec|
   else
     spec.add_dependency 'bunny', '>= 1.5.0'
   end
-  spec.add_dependency 'lifeguard'
+  spec.add_dependency 'lifeguard', '>= 0.0.9'
   spec.add_dependency 'middleware'
   spec.add_dependency 'thor'
 

data/lib/action_subscriber.rb

@@ -81,6 +81,7 @@ module ActionSubscriber
         logger.info "    --    exchange: #{route.exchange}"
         logger.info "    --       queue: #{route.queue}"
         logger.info "    -- routing_key: #{route.routing_key}"
+        logger.info "    --  threadpool: #{route.threadpool.name}, pool_size: #{route.threadpool.pool_size}"
       end
     end
   end

data/lib/action_subscriber/babou.rb

@@ -96,8 +96,11 @@ module ActionSubscriber
       wait_loops = 0
       ::ActionSubscriber::Babou.stop_receving_messages!
 
-      while ::ActionSubscriber::Threadpool.pool.busy_size > 0 && wait_loops < ::ActionSubscriber.configuration.seconds_to_wait_for_graceful_shutdown
-        puts "waiting for threadpool to empty (#{::ActionSubscriber::Threadpool.pool.busy_size})"
+      while ::ActionSubscriber::Threadpool.busy? && wait_loops < ::ActionSubscriber.configuration.seconds_to_wait_for_graceful_shutdown
+        puts "waiting for threadpools to empty:"
+        ::ActionSubscriber::Threadpool.pools.each do |name, pool|
+          puts "  -- #{name} (remaining: #{pool.busy_size})" if pool.busy_size > 0
+        end
         Thread.pass
         wait_loops = wait_loops + 1
         sleep 1

data/lib/action_subscriber/bunny/subscriber.rb

@@ -17,6 +17,9 @@ module ActionSubscriber
         times_to_pop = [::ActionSubscriber::Threadpool.ready_size, ::ActionSubscriber.config.times_to_pop].min
         times_to_pop.times do
           queues.each do |route, queue|
+            # Handle busy checks on a per threadpool basis
+            next if route.threadpool.busy?
+
             delivery_info, properties, encoded_payload = queue.pop(route.queue_subscription_options)
             next unless encoded_payload # empty queue
             ::ActiveSupport::Notifications.instrument "popped_event.action_subscriber", :payload_size => encoded_payload.bytesize, :queue => queue.name

data/lib/action_subscriber/march_hare/subscriber.rb

@@ -13,6 +13,9 @@ module ActionSubscriber
         times_to_pop = [::ActionSubscriber::Threadpool.ready_size, ::ActionSubscriber.config.times_to_pop].min
         times_to_pop.times do
           queues.each do |route,queue|
+            # Handle busy checks on a per threadpool basis
+            next if route.threadpool.busy?
+
             metadata, encoded_payload = queue.pop(route.queue_subscription_options)
             next unless encoded_payload
             ::ActiveSupport::Notifications.instrument "popped_event.action_subscriber", :payload_size => encoded_payload.bytesize, :queue => queue.name

data/lib/action_subscriber/threadpool.rb

@@ -4,15 +4,14 @@ module ActionSubscriber
     # Class Methods
     #
     def self.busy?
-      pools.any? do |_pool_name, pool|
-        pool.pool_size == pool.busy_size
-      end
+      !ready?
     end
 
     def self.new_pool(name, pool_size = nil)
       fail ArgumentError, "#{name} already exists as a threadpool" if pools.key?(name)
       pool_size ||= ::ActionSubscriber.config.threadpool_size
       pools[name] = ::Lifeguard::InfiniteThreadpool.new(
+        :name => name,
         :pool_size => pool_size
       )
     end
@@ -24,13 +23,14 @@ module ActionSubscriber
     def self.pools
       @pools ||= {
         :default => ::Lifeguard::InfiniteThreadpool.new(
+          :name => :default,
           :pool_size => ::ActionSubscriber.config.threadpool_size
         )
       }
     end
 
     def self.ready?
-      !busy?
+      pools.any? { |_pool_name, pool| !pool.busy? }
     end
 
     def self.ready_size

data/lib/action_subscriber/version.rb

@@ -1,3 +1,3 @@
 module ActionSubscriber
-  VERSION = "2.1.1"
+  VERSION = "2.1.2"
 end

data/spec/lib/action_subscriber/threadpool_spec.rb

@@ -1,32 +1,46 @@
 describe ::ActionSubscriber::Threadpool do
   describe "busy?" do
-    context "when the workers are busy" do
+    context "when the pool is busy" do
       it "returns true" do
-        allow(::ActionSubscriber::Threadpool.pool).to receive(:busy_size).and_return(8)
-        expect(::ActionSubscriber::Threadpool.busy?).to eq(true)
+        allow(::ActionSubscriber::Threadpool).to receive(:ready?).and_return(false)
+        expect(::ActionSubscriber::Threadpool).to be_busy
       end
     end
 
-    context "when there are idle workers" do
+    context "when the pool is not busy" do
       it "returns false" do
-        allow(::ActionSubscriber::Threadpool.pool).to receive(:busy_size).and_return(1)
-        expect(::ActionSubscriber::Threadpool.busy?).to eq(false)
+        allow(::ActionSubscriber::Threadpool).to receive(:ready?).and_return(true)
+        expect(::ActionSubscriber::Threadpool).to_not be_busy
       end
     end
   end
 
   describe "ready?" do
-    context "when the pool is busy" do
+    context "when all the workers are full" do
       it "returns false" do
-        allow(::ActionSubscriber::Threadpool).to receive(:busy?).and_return true
-        expect(::ActionSubscriber::Threadpool.ready?).to eq(false)
+        ::ActionSubscriber::Threadpool.new_pool(:some_dumb_pool, 2)
+
+        ::ActionSubscriber::Threadpool.pools.map do |_name, pool|
+          allow(pool).to receive(:busy_size).and_return(pool.pool_size)
+        end
+
+        expect(::ActionSubscriber::Threadpool).to_not be_ready
       end
     end
 
-    context "when the pool is not busy" do
+    context "when only one of the workers is full" do
       it "returns true" do
-        allow(::ActionSubscriber::Threadpool).to receive(:busy?).and_return false
-        expect(::ActionSubscriber::Threadpool.ready?).to eq(true)
+        pool = ::ActionSubscriber::Threadpool.new_pool(:some_other_dumb_pool, 2)
+        allow(pool).to receive(:busy_size).and_return(2)
+
+        expect(::ActionSubscriber::Threadpool).to be_ready
+      end
+    end
+
+    context "when there are idle workers" do
+      it "returns true" do
+        allow(::ActionSubscriber::Threadpool.pool).to receive(:busy_size).and_return(1)
+        expect(::ActionSubscriber::Threadpool).to be_ready
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: action_subscriber
 version: !ruby/object:Gem::Version
-  version: 2.1.1
+  version: 2.1.2
 platform: java
 authors:
 - Brian Stien
@@ -12,7 +12,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2016-02-19 00:00:00.000000000 Z
+date: 2016-02-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
@@ -47,7 +47,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.0.9
   name: lifeguard
   prerelease: false
   type: :runtime
@@ -55,7 +55,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.0.9
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements: