perfectqueue 0.8.19 → 0.8.20

data/ChangeLog

@@ -1,4 +1,10 @@
 
+== 2012-09-08 version 0.8.20
+
+* Show log messages when a worker decides next status of a task (finished!,
+  release!, release! and cancel_request!)
+
+
 == 2012-09-04 version 0.8.19
 
 * rdb_compat backend uses >= 0 condition instead of IS NOT NULL to work with

data/lib/perfectqueue/multiprocess/thread_processor.rb

@@ -123,7 +123,7 @@ module PerfectQueue
       end
 
       def process(task)
-        @log.info "acquired task id=#{@processor_id}: #{task.inspect}"
+        @log.info "acquired task task=#{task.key} id=#{@processor_id}: #{task.inspect}"
         begin
           r = @runner.new(task)
           @tm.set_task(task, r)
@@ -132,8 +132,9 @@ module PerfectQueue
           ensure
             @tm.task_finished(task)
           end
+          @log.info "completed processing task=#{task.key} id=#{@processor_id}:"
         rescue
-          @log.error "process failed id=#{@processor_id}: #{$!.class}: #{$!}"
+          @log.error "unexpectedly failed task=#{task.key} id=#{@processor_id}: #{$!.class}: #{$!}"
           $!.backtrace.each {|bt| @log.warn "\t#{bt}" }
           raise  # force exit
         end

data/lib/perfectqueue/task_monitor.rb

@@ -54,6 +54,7 @@ module PerfectQueue
 
     def set_task(task, runner)
       task.extend(TaskMonitorHook)
+      task.log = @log
       task.task_monitor = self
       task.runner = runner
       @mutex.synchronize {
@@ -97,7 +98,7 @@ module PerfectQueue
     # callback
     def task_finished(task, &block)
       @mutex.synchronize {
-        ret = block.call if block
+        ret = block.call if block  # TODO is this ought to be synchronized?
         if task == @task
           @task = nil
         end
@@ -152,6 +153,7 @@ module PerfectQueue
   end
 
   module TaskMonitorHook
+    attr_accessor :log
     attr_accessor :task_monitor
     attr_accessor :runner
 
@@ -164,24 +166,28 @@ module PerfectQueue
     attr_reader :heartbeat_message
 
     def finish!(*args, &block)
+      @log.info "finished task=#{self.key}" if @log
       @task_monitor.task_finished(self) {
         super(*args, &block)
       }
     end
 
     def release!(*args, &block)
+      @log.info "release task=#{self.key}" if @log
       @task_monitor.task_finished(self) {
         super(*args, &block)
       }
     end
 
     def retry!(*args, &block)
+      @log.info "retry task=#{self.key}" if @log
       @task_monitor.task_finished(self) {
         super(*args, &block)
       }
     end
 
     def cancel_request!(*args, &block)
+      @log.info "cancel request task=#{self.key}" if @log
       @task_monitor.task_finished(self) {
         super(*args, &block)
       }

data/lib/perfectqueue/version.rb

@@ -1,3 +1,3 @@
 module PerfectQueue
-  VERSION = "0.8.19"
+  VERSION = "0.8.20"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: perfectqueue
 version: !ruby/object:Gem::Version
-  version: 0.8.19
+  version: 0.8.20
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-10-05 00:00:00.000000000 Z
+date: 2012-10-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sequel