chore-core 1.8.1 → 1.8.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e8debf965223bd61c1b247c0c275c3a4f758085f
-  data.tar.gz: 51b646120ff568753112859186b6714a3846a819
+  metadata.gz: f1a706f2f8f8eefe36f16cfaea8636f6fe13df8d
+  data.tar.gz: d4c0b0f97c85916e509ec609bd9a9e38cac6aac0
 SHA512:
-  metadata.gz: 6b2b1f4e194ea2704734b212255135f32c504937b4152e6bc490a25401861d34da64f38e0a8e2ca2f8c7b0be5a5f1a45dd8553fbcbcf7aa0a86028c569ac5b00
-  data.tar.gz: 264df2f96cc45d6a83a48688b602c96ee4bb2df19ad23b3d2a26c2476276ee8545798ece8e94bda0edf1a6d338e35d10fe9718d72e1ca03ff3ecfe970cb0b85a
+  metadata.gz: c92dcbc42098e17b5e1d3ac05860ca1b74022725a2839964f6480bc8c3f080555e950757997ad9d3d4cdbd4b122e02e467614bb481a4108dfe65c5a74b797c6c
+  data.tar.gz: 95baf93fe723268b499447c24464443e3e7e19ed62d5718394c7dae66bcafb44a33f47133b3973827efd5856acc4fee2aab795adc33515c18068807f9e5039aa

data/lib/chore/queues/filesystem/publisher.rb

@@ -24,10 +24,13 @@ module Chore
               if f.flock(File::LOCK_EX | File::LOCK_NB) && f.size == 0
                 begin
                   f.write(job.to_json)
+                rescue StandardError => e
+                  Chore.logger.error "#{e.class.name}: #{e.message}. Could not write #{job[:class]} job to '#{queue_name}' queue file."
+                  Chore.logger.error e.backtrace.join("\n")
                 ensure
                   f.flock(File::LOCK_UN)
-                  break
                 end
+                break
               end
             end
           end

data/lib/chore/strategies/worker/single_worker_strategy.rb

@@ -21,16 +21,27 @@ module Chore
         worker.stop! if worker
       end
 
-      # Assigns work if there isn't already a worker in progress. Otherwise, is a noop
+      # Assigns work if there isn't already a worker in progress. In this, the
+      # single worker strategy, this should never be called if the worker is in
+      # progress.
       def assign(work)
         if workers_available?
-          @worker = Worker.new(work, @options)
-          @worker.start
-          @worker = nil
-          true
+          begin
+            @worker = worker_klass.new(work, @options)
+            @worker.start
+            true
+          ensure
+            @worker = nil
+          end
+        else
+          Chore.logger.error { "#{self.class}#assign: single worker is unavailable, but assign has been re-entered: #{caller * "\n"}" }
         end
       end
 
+      def worker_klass
+        Worker
+      end
+
       # Returns true if there is currently no worker
       def workers_available?
         @worker.nil?

data/lib/chore/version.rb

@@ -2,7 +2,7 @@ module Chore
   module Version #:nodoc:
     MAJOR = 1
     MINOR = 8
-    PATCH = 1
+    PATCH = 2
 
     STRING = [ MAJOR, MINOR, PATCH ].join('.')
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: chore-core
 version: !ruby/object:Gem::Version
-  version: 1.8.1
+  version: 1.8.2
 platform: ruby
 authors:
 - Tapjoy
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-01-22 00:00:00.000000000 Z
+date: 2016-02-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json