asynchronic 3.0.0 → 3.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: fe064d93526974fae713afcd2d5fc4730cb65b94
-  data.tar.gz: f6ca2831a8d78323370116397d1cbb9d48c66837
+  metadata.gz: 0e0f25f70fa41da06affe41aa2fb639b7bfd542a
+  data.tar.gz: 0a59f33dece383cd47df04ee457221101bbed618
 SHA512:
-  metadata.gz: a6668033166027a2d54aca7b094f8f90642f4b180f01299e99099a62b47f3dffd871799773248c3ad0bb3a4127885029234cda953265782926b685a5596743b3
-  data.tar.gz: 86da5ba9d7207fae1d6f01f77f433f8ed7e3b6d6e64e3bec177755088ee0c844f5c8a768f942c905ce730d9ea1d6dfcf5caee4db1a83b20bbf4a7727a721ed04
+  metadata.gz: 80d20d5b4ca50a04ed124b03d7b76000847a433877b45bb7bd2dc77e22f0984ec6f4215af76bd5a881a5bd550b67725a3a1d856698f96d878f9f55725e181c67
+  data.tar.gz: 51c7ed3e8686731277c31fbdea1bcb2bfc62c10959fbbcab7cf46a4d0cf6dc7f7ff2c6063eb5eea4809aeed773a150593787b1fdef6ca1d7afef8cf2c4531ed1

data/lib/asynchronic.rb

@@ -24,8 +24,8 @@ module Asynchronic
   attr_config :retry_timeout, 30
   attr_config :garbage_collector_timeout, 30
   attr_config :redis_data_store_sync_timeout, 0.01
-  attr_config :keep_alive_timeout, 0.1
-  attr_config :connection_name, "HOST=#{Socket.gethostname},PID=#{::Process.pid}"
+  attr_config :keep_alive_timeout, 1
+  attr_config :connection_name, "HOST=#{Socket.gethostname},PID=#{::Process.pid},UUID=#{SecureRandom.uuid}"
 
   def self.environment
     Environment.new queue_engine, data_store, notifier

data/lib/asynchronic/data_store/redis.rb

@@ -12,7 +12,7 @@ module Asynchronic
       end
 
       def [](key)
-        value = @redis.call 'GET', @scope[key]
+        value = @redis.call! 'GET', @scope[key]
         value ? Marshal.load(value) : nil
       rescue => ex
         Asynchronic.logger.warn('Asynchronic') { ex.message }
@@ -20,29 +20,29 @@ module Asynchronic
       end
 
       def []=(key, value)
-        @redis.call 'SET', @scope[key], Marshal.dump(value)
+        @redis.call! 'SET', @scope[key], Marshal.dump(value)
       end
 
       def delete(key)
-        @redis.call 'DEL', @scope[key]
+        @redis.call! 'DEL', @scope[key]
       end
 
       def delete_cascade(key)
-        @redis.call 'DEL', @scope[key]
-        @redis.call('KEYS', @scope[key]['*']).each { |k| @redis.call 'DEL', k }
+        @redis.call! 'DEL', @scope[key]
+        @redis.call!('KEYS', @scope[key]['*']).each { |k| @redis.call! 'DEL', k }
       end
 
       def keys
-        @redis.call('KEYS', @scope['*']).map { |k| Key[k].remove_first }
+        @redis.call!('KEYS', @scope['*']).map { |k| Key[k].remove_first }
       end
 
       def synchronize(key)
-        while @redis.call('GETSET', @scope[key][LOCKED], LOCKED) == LOCKED
+        while @redis.call!('GETSET', @scope[key][LOCKED], LOCKED) == LOCKED
           sleep Asynchronic.redis_data_store_sync_timeout
         end
         yield
       ensure
-        @redis.call 'DEL', @scope[key][LOCKED]
+        @redis.call! 'DEL', @scope[key][LOCKED]
       end
 
       def connection_args

data/lib/asynchronic/process.rb

@@ -267,6 +267,9 @@ module Asynchronic
 
     def connected?
       connection_name && environment.queue_engine.active_connections.include?(connection_name)
+    rescue => ex
+      Asynchronic.logger.error('Asynchronic') { "#{ex.message}\n#{ex.backtrace.join("\n")}" }
+      true
     end
 
   end

data/lib/asynchronic/queue_engine/ost.rb

@@ -16,12 +16,12 @@ module Asynchronic
       end
 
       def queues
-        (@queues.values.map(&:key) | redis.call('KEYS', 'ost:*')).map { |q| q.to_s[4..-1].to_sym }
+        (@queues.values.map(&:key) | redis.call!('KEYS', 'ost:*')).map { |q| q.to_s[4..-1].to_sym }
       end
 
       def clear
         @queues.clear
-        redis.call('KEYS', 'ost:*').each { |k| redis.call('DEL', k) }
+        redis.call!('KEYS', 'ost:*').each { |k| redis.call!('DEL', k) }
       end
 
       def listener
@@ -33,9 +33,10 @@ module Asynchronic
       end
 
       def active_connections
-        redis.call('CLIENT', 'LIST').split("\n").map do |connection_info|
-          connection_info.split(' ').detect { |a| a.match(/name=/) }[5..-1]
-        end.uniq.reject(&:empty?)
+        redis.call!('CLIENT', 'LIST').split("\n").map do |connection_info|
+          name_attr = connection_info.split(' ').detect { |a| a.match(/name=/) }
+          name_attr ? name_attr[5..-1] : nil
+        end.uniq.compact.reject(&:empty?)
       end
 
       private
@@ -43,7 +44,7 @@ module Asynchronic
       def notify_keep_alive
         Thread.new do
           loop do
-            redis.call 'CLIENT', 'SETNAME', Asynchronic.connection_name
+            redis.call! 'CLIENT', 'SETNAME', Asynchronic.connection_name
             sleep Asynchronic.keep_alive_timeout
           end
         end
@@ -58,11 +59,11 @@ module Asynchronic
         end
 
         def pop
-          redis.call 'RPOP', key
+          redis.call! 'RPOP', key
         end
 
         def empty?
-          redis.call('EXISTS', key) == 0
+          redis.call!('EXISTS', key) == 0
         end
 
         def size

data/lib/asynchronic/version.rb

@@ -1,3 +1,3 @@
 module Asynchronic
-  VERSION = '3.0.0'
+  VERSION = '3.0.1'
 end

data/spec/facade_spec.rb

@@ -69,11 +69,11 @@ describe Asynchronic, 'Facade' do
   end
 
   it 'Keep alive timeout' do
-    Asynchronic.keep_alive_timeout.must_equal 0.1
+    Asynchronic.keep_alive_timeout.must_equal 1
   end
 
   it 'Connection name' do
-    Asynchronic.connection_name.must_equal "HOST=#{Socket.gethostname},PID=#{::Process.pid}"
+    Asynchronic.connection_name.must_match /^HOST=#{Socket.gethostname},PID=#{::Process.pid}/
   end
 
 end

data/spec/jobs.rb

@@ -200,7 +200,7 @@ end
 class WithRetriesJob < Asynchronic::Job
   def call
     @counter = 0
-    retry_when [RuntimeError] do
+    retry_when [RuntimeError], 0.1 do
       @counter += 1
       raise 'Counter < 3' if @counter < 3
       @counter

data/spec/process/life_cycle_examples.rb

@@ -29,8 +29,13 @@ module LifeCycleExamples
 
     process.must_have_connection_name
     process.wont_be :dead?
+    
     process.send(:connected?).must_be_true
 
+    env.queue_engine.stub(:active_connections, ->() { raise 'Forced error' }) do
+      process.send(:connected?).must_be_true
+    end
+
     with_retries do
       events.last.must_equal process.status
       process.finalized?.must_equal is_finalized

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: asynchronic
 version: !ruby/object:Gem::Version
-  version: 3.0.0
+  version: 3.0.1
 platform: ruby
 authors:
 - Gabriel Naiman
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-06-24 00:00:00.000000000 Z
+date: 2019-07-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ost