xpool 0.10.1 → 0.11.0

data/ChangeLog.txt

@@ -1,4 +1,4 @@
-== HEAD
+== v0.11.0
 	- no longer log from trap(…) block.
 	the logger cannot log from within a trap block on Ruby2, so I've removed 
 	logging when a subprocess gets a request to shutdown.

data/README.md

@@ -29,7 +29,7 @@ pool with five subprocesses and schedule a unit of work five times, each
 subprocess in the pool would have executed the unit of work once.
 
 A pool can become "dry" whenever all its subprocesses are busy. If you schedule
-a unit of work on a dry pool the same scheduling logic apllies but instead of
+a unit of work on a dry pool the same scheduling logic applies but instead of
 the unit of work executing right away it will be executed whenever the 
 assigned subprocess is no longer busy. It is also possible to query the pool 
 and ask if it is dry, but you can also ask an individual subprocess if it is
@@ -109,27 +109,6 @@ pool.broadcast Unit.new
 pool.shutdown
 ```
 
-__DEBUGGING OUTPUT__
-
-xpool can print helpful debugging information if you set `XPool.debug` 
-to true:
-
-```ruby
-XPool.debug = true
-```
-
-Or you can temporarily enable debugging output for the duration of a block:
-
-```ruby
-XPool.debug do 
-  pool = XPool.new 2
-  pool.shutdown
-end
-```
-
-The debugging information you'll see is all about how the pool is operating. 
-It can be interesting to look over even if you're not bug hunting.
-
 __SIGUSR1__
 
 All xpool managed subprocesses define a signal handler for the SIGUSR1 signal.

data/docs/debugging.md

@@ -0,0 +1,18 @@
+__DEBUGGING__
+
+xpool can print helpful debugging information if you set `XPool.debug` 
+to true. The debugging information you'll see is all about how the pool is 
+operating. It can be interesting to look over even if you're not bug hunting.
+
+```ruby
+XPool.debug = true
+```
+
+Or you can temporarily enable debugging output for the duration of a block:
+
+```ruby
+XPool.debug do 
+  pool = XPool.new 2
+  pool.shutdown
+end
+```

data/lib/xpool.rb

@@ -239,11 +239,6 @@ private
   end
 
   def _resize(new_size, with_force)
-    if Range === new_size
-      warn "[DEPRECATED] XPool#resize! no longer accepts a Range." \
-           "Please use a Fixnum instead."
-      new_size = range.to_a.size
-    end
     new_size -= 1
     old_size = size - 1
     if new_size == old_size

data/lib/xpool/process.rb

@@ -164,22 +164,22 @@ private
 
   def spawn
     fork do
-      trap :SIGUSR1 do
-        XPool.log "#{::Process.pid} got request to shutdown."
-        @shutdown_requested = true
-      end
+      trap(:SIGUSR1) { @shutdown_requested = true }
       loop &method(:read_loop)
     end
   end
 
   def read_loop
     if @channel.readable?
+      @frequency += 1
       @s_channel.put busy: true
       msg = @channel.get
+      msg[:unit].setup if @frequency == 1 && msg[:unit].respond_to?(:setup)
       msg[:unit].run *msg[:args]
       @s_channel.put busy: false
+    else
+      sleep 0.05
     end
-    sleep 0.05
   rescue Exception => e
     @s_channel.put failed: true, dead: true, backtrace: e.backtrace
     XPool.log "Process with ID '#{@id}' has failed."

data/lib/xpool/version.rb

@@ -1,3 +1,3 @@
 class XPool
-  VERSION = "0.10.1"
+  VERSION = "0.11.0"
 end

data/test/xpool_process_test.rb

@@ -14,7 +14,7 @@ class XPoolProcessTest < Test::Unit::TestCase
     io_writers.each { |writer| @process.schedule writer }
     until @process.failed?; sleep 0.1; end
     @process.restart
-    sleep 0.5
+    sleep 0.1
     io_writers.each { |io_writer| assert io_writer.wrote_to_disk? }
   end
 
@@ -26,6 +26,16 @@ class XPoolProcessTest < Test::Unit::TestCase
     refute @process.busy?, 'Expected process to not be busy'
   end
 
+  def test_call_of_setup
+    writer = IOSetupWriter.new
+    @process.schedule writer
+    sleep 0.1
+    assert writer.wrote_to_disk?
+    @process.schedule writer
+    sleep 0.1
+    refute writer.wrote_to_disk?
+  end
+
   def test_busy_on_exception
     @process.schedule Raiser.new
     sleep 0.1

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: xpool
 version: !ruby/object:Gem::Version
-  version: 0.10.1
+  version: 0.11.0
   prerelease: 
 platform: ruby
 authors:
@@ -45,6 +45,7 @@ files:
 - Rakefile
 - bench/.gitkeep
 - bench/pool-schedule.rb
+- docs/debugging.md
 - docs/unhandled_exceptions.md
 - lib/xpool.rb
 - lib/xpool/process.rb
@@ -70,7 +71,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -3457064609731373945
+      hash: 3959300438831349653
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -79,7 +80,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -3457064609731373945
+      hash: 3959300438831349653
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.23