postburner 0.6.1 → 0.6.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ca9bed4afd8b585d14934b4ba09b20c58109d8de39d5300d135069e2723c3aba
-  data.tar.gz: 5ae5f4a07e53ac69b4d4b795f3b8c8bdf5b9a087e1a8513f6fed03cb47e48210
+  metadata.gz: 6d47898f4be7a93d1643ba6ff32db238b341a2ceed09bbc18de2774d3048d9cd
+  data.tar.gz: 934f5a102f6644e679c2bf30d4708e171f01d88b5652fe6bcef2cac8b6fe4c50
 SHA512:
-  metadata.gz: c741884d62b1b71b7c8ce928baff5fef35054c58962f530ee06764b4947477ca394712300256e98409187494092c7164a5f8490482e54a19d0f1776b8465f783
-  data.tar.gz: 193a2e625139d0a441503afc15d64888eb25234d0ae81ca320a319506130e8eb1c960141d60b97066633f6a777ab47fe21d49d42bc448497b4225e8c78924051
+  metadata.gz: cb29ebe178b4675552a441c335fd95867a9a26c0a15235c42d5dff0f8ab72c40c417c728bc171ecf461ef3e78c9672364b89fae3b91419dceac59f80ad4c9986
+  data.tar.gz: 807cb26488522a5a3becdc53fbc43b44a9123d1355ee31d2b79dc2d1d479ce714a6b3a5f20c60b74e2f2fc22753736e6fde9822fe59e83feab69024e34e60deb

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## v0.6.2 - 2021-11-01
+
+### Fixed
+- add #requeue method.
+- check if AlreadyProcessed.
+
 ## v0.6.1 - 2021-11-01
 
 ### Updated

data/app/models/postburner/job.rb

@@ -31,15 +31,19 @@ module Postburner
     def queue!(options={})
       return if self.queued_at.present? && self.bkid.present?
       raise ActiveRecord::RecordInvalid, "Can't queue unless valid." unless self.valid?
+      raise AlreadyProcessed, "Processed at #{self.processed_at}" if self.processed_at
+
       at = options.delete(:at)
+      now = Time.zone.now
 
+      self.queued_at = now
       self.run_at = case
                     when at.present?
                       # this is rudimentary, add error handling
-                      options[:delay] ||= at.to_i - Time.zone.now.to_i
+                      options[:delay] ||= at.to_i - now.to_i
                       at
                     when options[:delay].present?
-                      Time.zone.now + options[:delay].seconds
+                      now + options[:delay].seconds
                     end
 
       @_insert_options = options
@@ -47,6 +51,13 @@ module Postburner
       self.save!
     end
 
+    def requeue!(options={})
+      self.delete!
+      self.bkid, self.queued_at = nil, nil
+
+      self.queue! options
+    end
+
     def will_insert?
       @_insert_options.is_a? Hash
     end
@@ -104,7 +115,7 @@ module Postburner
           return
         end
 
-        self.log!('START')
+        self.log!("START (bkid #{self.bkid})")
 
         begin
           self.perform(args)
@@ -114,7 +125,7 @@ module Postburner
           raise exception
         end
 
-        self.log!('DONE')
+        self.log!("DONE (bkid #{self.bkid})")
 
         begin
           now = Time.zone.now
@@ -181,6 +192,7 @@ module Postburner
       self.errata << [
         Time.zone.now,
         {
+          bkid:       self.bkid,
           class:      exception.class,
           message:    exception.message,
           backtrace:  exception.backtrace,
@@ -200,9 +212,10 @@ module Postburner
       self.logs << [
         Time.zone.now, # time
         {
-          level: options[:level], # level
-          message: message, # message
-          elapsed: self.elapsed_ms, # ms from start
+          bkid:     self.bkid,
+          level:    options[:level], # level
+          message:  message, # message
+          elapsed:  self.elapsed_ms, # ms from start
         }
       ]
     end
@@ -219,6 +232,12 @@ module Postburner
       self.update_column :logs, self.logs
     end
 
+    def intended_at
+      self.run_at ? self.run_at : self.queued_at
+    end
+
+    class AlreadyProcessed < StandardError; end
+
     private
 
     def persist_metadata!(data={})
@@ -236,19 +255,22 @@ module Postburner
     end
 
     def insert!(options={})
-      response = Backburner::Worker.enqueue(
-        Postburner::Job,
-        self.id,
-        options
-      )
+      response = nil
+
+      Job.transaction do
+        response = Backburner::Worker.enqueue(
+          Postburner::Job,
+          self.id,
+          options
+        )
+
+        persist_metadata!(
+          bkid: response[:id],
+        )
+      end
 
       self.log("QUEUED: #{response}")
 
-      persist_metadata!(
-        queued_at:  Time.zone.now,
-        bkid:       response[:id],
-      )
-
       response
     end
 
@@ -256,14 +278,10 @@ module Postburner
       now = Time.zone.now
       self.attempts << now
       self.attempting_at ||= now
-      self.lag ||= (self.attempting_at - self.intended_at) * 1000
+      self.lag ||= (self.attempting_at - self.intended_at) * 1000 rescue nil
       now
     end
 
-    def intended_at
-      self.run_at ? self.run_at : self.queued_at
-    end
-
     def ensure_sid!
       self.sid ||= SecureRandom.uuid
     end

data/lib/postburner/version.rb

@@ -1,3 +1,3 @@
 module Postburner
-  VERSION = '0.6.1'
+  VERSION = '0.6.2'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: postburner
 version: !ruby/object:Gem::Version
-  version: 0.6.1
+  version: 0.6.2
 platform: ruby
 authors:
 - Matt Smith