unique_job 0.1.0 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: af33cb02d39f50c4d16316374ee2079b863505321370817e1df1a293d5b6674f
-  data.tar.gz: '02906948d9857c66215b6984ff2966e766bde452eb375f08c71f01739b3dbf25'
+  metadata.gz: c480f4b0fe78931cf109fbca3db98032cd5bc2e1ad065dc218c56e1ac00361b5
+  data.tar.gz: e67b02dd3bde8bfb46d417e46bac20780f7c05eb67dcd44f93d9fda831ecde15
 SHA512:
-  metadata.gz: 6e53fde574f9343471ccdbbf72a9a3562e4a8c95950968ea55f1cbb5f774989b8ab5f781dd129f299817d4e45dbfaa9fe4ae42b3aa9b5b86dd4475c4d6d6ab50
-  data.tar.gz: cd1f0aff1da071006a5c9c003882650bfb2d534b0c1bc455469fddfdc3de60b889e404ddb072426f155ea944315bcd88c9e0ecb24009b0e011fdd4c878b30347
+  metadata.gz: 44785d034535d444ee219a7b0ad0dd064b02b85aee5289f13ba74a1c2d4fded15898ed2b974c4be4fbc5d6644bf4ba4e704a461c2f3f96b8341e93ea9aeb6dbc
+  data.tar.gz: bd9b4b1ef9d56ae58f724d1dc827f3747895154b0f7fd978dfd1739e02fdb547fb1d3ee1d2ea5d667e92b572ad1bc1448e579fcd181b7da53acd1c811ab16f55

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    unique_job (0.1.0)
+    unique_job (0.2.0)
       redis
       sidekiq (> 6.0, < 7.0)
 

data/lib/unique_job/job_history.rb

@@ -1,7 +1,9 @@
+require 'unique_job/logging'
+
 module UniqueJob
   class JobHistory
     def initialize(worker_class, queueing_class, ttl)
-      @key = "#{self.class}:#{queueing_class}:#{worker_class}"
+      @key = "#{self.class}:#{queueing_class.name.split('::')[-1]}:#{worker_class}"
       @ttl = ttl
     end
 
@@ -20,7 +22,7 @@ module UniqueJob
     end
 
     def exists?(val)
-      redis.exists(key(val))
+      redis.exists?(key(val))
     end
 
     def add(val)
@@ -44,18 +46,20 @@ module UniqueJob
     end
 
     module RescueAllRedisErrors
+      include Logging
+
       %i(
         ttl
         exists?
         add
       ).each do |method_name|
         define_method(method_name) do |*args, &blk|
-          start = Time.zone.now
+          start = Time.now
           super(*args, &blk)
         rescue => e
-          elapsed = Time.zone.now - start
-          Rails.logger.warn "Rescue all errors in #{self.class}##{method_name} #{e.inspect} elapsed=#{sprintf("%.3f sec", elapsed)}"
-          Rails.logger.debug { e.backtrace.join("\n") }
+          elapsed = Time.now - start
+          logger.warn "Rescue all errors in #{self.class}##{method_name} #{e.inspect} elapsed=#{sprintf("%.3f sec", elapsed)}"
+          logger.debug { e.backtrace.join("\n") }
           nil
         end
       end

data/lib/unique_job/logging.rb

@@ -0,0 +1,17 @@
+require 'logger'
+
+module UniqueJob
+  module Logging
+    def logger
+      if File.basename($0) == 'rake'
+        Logger.new(STDOUT, level: Logger::WARN)
+      elsif defined?(Sidekiq)
+        Sidekiq.logger
+      elsif defined?(Rails)
+        Rails.logger
+      else
+        Logger.new(STDOUT, level: Logger::WARN)
+      end
+    end
+  end
+end

data/lib/unique_job/util.rb

@@ -1,11 +1,16 @@
+require 'unique_job/logging'
+require 'unique_job/job_history'
+
 module UniqueJob
   module Util
+    include Logging
+
     def perform_if_unique(worker, args, &block)
       if worker.respond_to?(:unique_key)
         unique_key = worker.unique_key(*args)
 
         if unique_key.nil? || unique_key.to_s.empty?
-          logger.warn { "#{class_name}##{__method__} Key is blank worker=#{worker.class} args=#{truncate(args.inspect)}" }
+          logger.warn { "#{self.class}##{__method__} Key is blank worker=#{worker.class} args=#{truncate(args.inspect)}" }
           yield
         elsif perform_unique_check(worker, args, unique_key.to_s)
           yield
@@ -19,7 +24,7 @@ module UniqueJob
       history = job_history(worker)
 
       if history.exists?(unique_key)
-        logger.info { "#{class_name}##{__method__} Skip duplicate job for #{history.ttl} seconds, remaining #{history.ttl(unique_key)} seconds worker=#{worker.class} args=#{truncate(args.inspect)}" }
+        logger.info { "#{self.class}##{__method__} Skip duplicate job for #{history.ttl} seconds, remaining #{history.ttl(unique_key)} seconds worker=#{worker.class} args=#{truncate(args.inspect)}" }
 
         perform_callback(worker, :after_skip, args)
 
@@ -32,7 +37,7 @@ module UniqueJob
 
     def job_history(worker)
       ttl = worker.respond_to?(:unique_in) ? worker.unique_in : 3600
-      JobHistory.new(worker.class, class_name, ttl)
+      JobHistory.new(worker.class, self.class, ttl)
     end
 
     def truncate(text, length: 100)
@@ -59,21 +64,5 @@ module UniqueJob
         end
       end
     end
-
-    def class_name
-      self.class.name.demodulize
-    end
-
-    def logger
-      if File.basename($0) == 'rake'
-        Logger.new(STDOUT, level: Logger::WARN)
-      elsif defined?(Sidekiq)
-        Sidekiq.logger
-      elsif defined?(Rails)
-        Rails.logger
-      else
-        Logger.new(STDOUT, level: Logger::WARN)
-      end
-    end
   end
 end

data/lib/unique_job/version.rb

@@ -1,3 +1,3 @@
 module UniqueJob
-  VERSION = "0.1.0"
+  VERSION = "0.2.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: unique_job
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - ts-3156
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-06-14 00:00:00.000000000 Z
+date: 2020-08-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sidekiq
@@ -64,6 +64,7 @@ files:
 - lib/unique_job.rb
 - lib/unique_job/client_middleware.rb
 - lib/unique_job/job_history.rb
+- lib/unique_job/logging.rb
 - lib/unique_job/server_middleware.rb
 - lib/unique_job/util.rb
 - lib/unique_job/version.rb