workers 0.6.0 → 0.6.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 50e412eae6d1bdf77fe6588ffc6a8d2412f739f7
-  data.tar.gz: ab20ac26b42dd524faa2d0d1171e6887de2e3bbf
+  metadata.gz: 8b6d69de3bb0d90b8d6ba9c1fd757f3efd8b6c25
+  data.tar.gz: 0c5911c5db1d7191abe82de486c49cbdbd331a6c
 SHA512:
-  metadata.gz: d22b5c5fb81b2b5b7105c1d237b2d9b779af7b2ef9cad993cd80c9b012a851390d851ec976497f51d87a7cbc2265d31e3712c4d2b2d2572da65c7ed8fa9fbec7
-  data.tar.gz: 3c44a7b8cc2e9943e919cb02e231f2d25511b5b24a4df1be847ef7e9c3b7ae083d07d35b3f1f99c25d5e8784745953300301ebecce7b67c155de3182ca6ecd94
+  metadata.gz: 8cc43d9d7779ec4fb95bea731fb584b85151f55198d7c3f3a274923acbc03b32b22b9626d99cccfe7ec214068306951daa33b60d1213e28e43c9f877ef733b74
+  data.tar.gz: 0c3d792a655461008fcfa6954c1a06c1fd27f62ef27cbd3d69f9c1bd9242096f9b986a6bd7cb83fbec52eb747021481e345822d905e14c44a2f4156857d22cd8

data/.ruby-version

@@ -1 +1 @@
-2.2.2
+2.4.1

data/.travis.yml

@@ -1,4 +1,6 @@
 language: ruby
 rvm:
-  - 2.2.2
-before_install: gem install bundler -v 1.10.5
+  - 2.4.0
+  - 2.3.3
+  - 2.2.6
+  - 2.1.10

data/README.md

@@ -109,7 +109,7 @@ This method uses a mutex so you can serialize portions of your tasks that aren't
     task = Workers::Task.new(
       :logger => nil,                   # Ruby logger instance.
       :on_perform => proc {},           # Required option.  Block of code to run.
-      :args => [],                      # Array of arguments passed to the 'perform' block.
+      :input => [],                     # Array of arguments passed to the 'perform' block.
       :on_finished => nil,              # Callback to execute after attempting to run the task.
       :max_tries => 1,                  # Number of times to try completing the task (without an exception).
     )

data/lib/workers/task_group.rb

@@ -42,7 +42,13 @@ module Workers
           @pool.perform { task.run }
         end
 
-        @conditional.wait(@internal_lock)
+        loop do
+          @conditional.wait(@internal_lock)
+          # The wait can return even if nothing called @conditional.signal,
+          # so we need to check to see if the condition actually changed.
+          # See https://github.com/chadrem/workers/issues/7
+          break if all_tasks_finished?
+        end
       end
 
       @tasks.all? { |t| t.succeeded? }
@@ -98,10 +104,14 @@ module Workers
     def finished(task)
       @internal_lock.synchronize do
         @finished_count += 1
-        @conditional.signal if @finished_count >= @tasks.count
+        @conditional.signal if all_tasks_finished?
       end
 
       nil
     end
+
+    def all_tasks_finished?
+      @finished_count >= @tasks.count
+    end
   end
 end

data/lib/workers/version.rb

@@ -1,3 +1,3 @@
 module Workers
-  VERSION = '0.6.0'
+  VERSION = '0.6.1'
 end

data/lib/workers/worker.rb

@@ -8,9 +8,9 @@ module Workers
     def initialize(options = {})
       @logger = Workers::LogProxy.new(options[:logger])
       @input_queue = options[:input_queue] || Queue.new
-      @thread = Thread.new { start_event_loop }
       @on_exception = options[:on_exception]
       @run = true
+      @thread = Thread.new { start_event_loop }
 
       nil
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: workers
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 0.6.1
 platform: ruby
 authors:
 - Chad Remesch
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2015-09-23 00:00:00.000000000 Z
+date: 2017-12-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -103,7 +103,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5
+rubygems_version: 2.6.11
 signing_key: 
 specification_version: 4
 summary: A Ruby gem for performing work in background threads.