unique_thread 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 84cdf521ad296d825e220ccea5b2610e70bef0dc
-  data.tar.gz: 07fe1de3059244ae0681c152fd249fdf75568d59
+  metadata.gz: 9c30eb2c2e3046dd457f030e2aff44def4aefdf3
+  data.tar.gz: 22ad34869f19e0aaf27d1bad6bc8a72299fc6ee6
 SHA512:
-  metadata.gz: 34a8c80ee3e585946fc6c4d662bf0ce1f525a4506880e5f585f081cc47cf68715d14c860060d5c86f7852d154fde9ac7cbbea1fcd4767bf047c0d1e2c3271f58
-  data.tar.gz: 2f7be9c25bfa9f6fded8c7d1fb9507f06790d43b6f784c8bfe41f563c63b8dd0f9a7e313063c0d18a9133fa91eefdc551769dbc8a6cf2cb69702956a190e30c9
+  metadata.gz: 9be00638892f61190a29bd1f42d85fb54bd5ba99d6a1590eef2298b68ba29537b5ee1f3546652c0ac5c363d205be3581c39e61f5ddf1ef19e3e203a55b292bf9
+  data.tar.gz: 20960248f2f34bb8d3b995dea6cad608b717013a5cfefe298146d3b671a7e9b41ea1a71ae9787c9cf48ef5a403be779abf247158283ec514a16903dbc7311267

data/lib/unique_thread.rb

@@ -7,7 +7,7 @@ require_relative 'unique_thread/locksmith'
 
 class UniqueThread
   class << self
-    attr_writer :logger, :redis
+    attr_writer :logger, :redis, :error_handlers
 
     def logger
       @logger ||= default_logger
@@ -17,6 +17,20 @@ class UniqueThread
       @redis ||= Redis.new
     end
 
+    def error_handlers
+      @error_handlers ||= []
+    end
+
+    def safe_thread
+      Thread.new do
+        begin
+          yield
+        rescue StandardError => error
+          report_error(error)
+        end
+      end
+    end
+
     private
 
     def default_logger
@@ -29,6 +43,11 @@ class UniqueThread
         Logger.new($stdout)
       end
     end
+
+    def report_error(exception)
+      logger.error(exception.inspect)
+      error_handlers.each { |handler| handler.call(exception) }
+    end
   end
 
   attr_reader :stopwatch, :locksmith
@@ -39,28 +58,22 @@ class UniqueThread
   end
 
   def run(&block)
-    safe_infinite_loop do
-      lock = locksmith.new_lock
-
-      if lock.acquired?
-        self.class.logger.info('Lock acquired! Running the unique thread.')
-        lock.while_held(&block)
-      else
-        self.class.logger.debug('Could not acquire the lock. Sleeping until next attempt.')
-        stopwatch.sleep_until_next_attempt(lock.locked_until.to_f)
-      end
+    self.class.safe_thread do
+      loop { try_being_the_unique_thread(&block) }
     end
   end
 
   private
 
-  def safe_infinite_loop
-    Thread.new do
-      begin
-        loop { yield }
-      rescue StandardError => error
-        self.class.logger.error(error)
-      end
+  def try_being_the_unique_thread(&block)
+    lock = locksmith.new_lock
+
+    if lock.acquired?
+      self.class.logger.info('Lock acquired! Running the unique thread.')
+      lock.while_held(&block)
+    else
+      self.class.logger.debug('Could not acquire the lock. Sleeping until next attempt.')
+      stopwatch.sleep_until_next_attempt(lock.locked_until.to_f)
     end
   end
 end

data/lib/unique_thread/locksmith.rb

@@ -70,7 +70,7 @@ class UniqueThread
     end
 
     def while_held
-      worker = Thread.new do
+      worker = UniqueThread.safe_thread do
         yield
         UniqueThread.logger.error('The blocked passed is not an infinite loop.')
       end

data/lib/unique_thread/stopwatch.rb

@@ -12,7 +12,6 @@ class UniqueThread
       Time.now.to_f
     end
 
-    # FIXME: Bad name. Lap maybe? Milestone?
     def next_renewal
       now + (downtime * 2 / 3)
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: unique_thread
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Fernando Seror