procrastinator 0.6.0 → 0.6.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 085c07da4d7c83dcccb46159c9bad1fc17717013
-  data.tar.gz: aaa2554edfe76056856da018705a21bef6a5345f
+  metadata.gz: 7162a00d304b7e97d03893a92fbe45b11534e305
+  data.tar.gz: e6e0fe48cf1cc1940aed6f85ccf889e36d40d0ff
 SHA512:
-  metadata.gz: 27cf0c1198f7c0f7e3c1208ff52f93a1c9fa7117ec2dcd8c7433f334cfeefb4b6e3dd43ed126be7058aeb60b7e78038fc3db138588e5eb267346fd7e43c7a0d6
-  data.tar.gz: cd0a15608dfa630cd70958d1040b06a63e127efca81e08f45642ee4fc427ead3a1ddf0d3f46243752f8a77d265cbcf562ecb2d01b51fed06d97ca1c9159bcfaf
+  metadata.gz: 3d23c3bba5d8df4963a7a33cbe70606dba64b86cba3f0c5861d37a9a878728e180bbea1afa196431fef7f2121e2a073cb2450989970a3b1c899378bd82f5a101
+  data.tar.gz: b2c35bafb8c18af846ab8199a923ef9bc9d09d81c731c3d9c0b46a7ca7f1ab1c87adf8f9f0ff1768341c86d9a59044b6f94db8230bcba6da15c68f0e849831df

data/lib/procrastinator/environment.rb

@@ -116,12 +116,12 @@ module Procrastinator
       def monitor_parent(worker)
          parent_pid = Process.ppid
 
-         heartbeat_thread = Thread.new(parent_pid) do |pid|
+         heartbeat_thread = Thread.new(parent_pid) do |ppid|
             loop do
                begin
-                  Process.kill(0, pid) # kill(0) will check if the process exists
+                  Process.kill(0, ppid) # kill with 0 flag checks if the process exists & has permissions
                rescue Errno::ESRCH
-                  worker.log_parent_exit
+                  worker.log_parent_exit(ppid: ppid, pid: Process.pid)
                   exit
                end
 

data/lib/procrastinator/queue_worker.rb

@@ -44,7 +44,7 @@ module Procrastinator
             end
          rescue StandardError => e
             @logger.fatal(e)
-               # raise e
+            # raise e
          end
       end
 
@@ -98,10 +98,16 @@ module Procrastinator
          end
       end
 
-      def log_parent_exit
+      # Logs a termination due to parent process termination
+      #
+      # == Parameters:
+      # @param ppid the parent's process id
+      # @param pid the child's process id
+      #
+      def log_parent_exit(ppid:, pid:)
          raise RuntimeError.new('Cannot log when logger not defined. Call #start_log first.') unless @logger
 
-         @logger.error("Terminated worker process (#{Process.pid}) due to main process (#{Process.ppid}) disappearing.")
+         @logger.error("Terminated worker process (pid=#{pid}) due to main process (ppid=#{ppid}) disappearing.")
       end
    end
 

data/lib/procrastinator/version.rb

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: procrastinator
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 0.6.1
 platform: ruby
 authors:
 - Robin Miller