postburner 1.0.0.pre.11 → 1.0.0.pre.12

data/app/concerns/postburner/commands.rb

@@ -136,7 +136,7 @@ module Postburner
     def remove!
       return if self.removed_at
       self.delete!
-      self.update_column(:removed_at, Time.zone.now)
+      self.update_column(:removed_at, Time.current)
     end
 
   end

data/app/concerns/postburner/execution.rb

@@ -23,17 +23,17 @@ module Postburner
       # queue strategy's {handle_perform!} method. This provides a unified API
       # for job execution regardless of strategy (async, test, or null).
       #
-      # Called automatically by Backburner workers in production. Can also be
+      # Called automatically by Postburner workers in production. Can also be
       # called manually for test/null strategies to trigger execution.
       #
       # @param id [Integer] Job ID to execute
-      # @param _ [Hash] Unused Backburner metadata parameter
+      # @param _ [Hash] Unused metadata parameter
       #
       # @return [void]
       #
-      # @example Backburner automatic execution (production)
-      #   # Jobs execute in tube: backburner.worker.queue.backburner-jobs
-      #   # Backburner calls: Postburner::Job.perform(job_id)
+      # @example Worker automatic execution (production)
+      #   # Jobs execute in configured Beanstalkd tube
+      #   # Worker calls: Postburner::Job.perform(job_id)
       #
       # @example Manual execution with NullQueue
       #   Postburner.null_strategy!
@@ -64,7 +64,7 @@ module Postburner
 
     # Executes the job with full lifecycle management and error handling.
     #
-    # This is the main execution method called by Backburner workers or test strategies.
+    # This is the main execution method called by Postburner workers or test strategies.
     # Performs validation checks, executes callbacks, calls the subclass {#perform} method,
     # tracks timing and statistics, and handles errors.
     #
@@ -100,7 +100,7 @@ module Postburner
           attempts:       self.attempts,
           attempt_count:  self.attempts.length,
           lag:            self.lag,
-          processing_at:  Time.zone.now,
+          processing_at:  Time.current,
         )
 
         begin
@@ -109,7 +109,7 @@ module Postburner
             return
           end
 
-          if self.queued_at > Time.zone.now
+          if self.queued_at > Time.current
             self.log! "Future Queued", level: :error
             return
           end
@@ -125,7 +125,7 @@ module Postburner
             return
           end
 
-          if self.run_at && self.run_at.to_i > Time.zone.now.to_i
+          if self.run_at && self.run_at.to_i > Time.current.to_i
             Postburner.queue_strategy.handle_premature_perform(self)
             return
           end
@@ -145,7 +145,7 @@ module Postburner
           self.log!("DONE (bkid #{self.bkid})")
 
           begin
-            now = Time.zone.now
+            now = Time.current
             _duration =  (now - self.processing_at) * 1000 rescue nil
 
             run_callbacks :processed do
@@ -180,7 +180,7 @@ module Postburner
     # @api private
     #
     def attempting
-      now = Time.zone.now
+      now = Time.current
       self.attempts << now
       self.attempting_at ||= now
       self.lag ||= (self.attempting_at - self.intended_at) * 1000 rescue nil

data/app/concerns/postburner/insertion.rb

@@ -66,7 +66,7 @@ module Postburner
       raise AlreadyProcessed, "Processed at #{self.processed_at}" if self.processed_at
 
       at = options.delete(:at)
-      now = Time.zone.now
+      now = Time.current
 
       self.queued_at = now
       self.run_at = case

data/app/concerns/postburner/logging.rb

@@ -61,7 +61,7 @@ module Postburner
     #
     def log_exception(exception)
       self.errata << [
-        Time.zone.now,
+        Time.current,
         {
           bkid:       self.bkid,
           class:      exception.class,
@@ -123,7 +123,7 @@ module Postburner
       options[:level] = :error unless LOG_LEVELS.member?(options[:level])
 
       self.logs << [
-        Time.zone.now, # time
+        Time.current, # time
         {
           bkid:     self.bkid,
           level:    options[:level], # level

data/app/concerns/postburner/statistics.rb

@@ -99,7 +99,7 @@ module Postburner
     #
     def elapsed_ms
       return unless self.attempting_at
-      (Time.zone.now - self.attempting_at) * 1000
+      (Time.current - self.attempting_at) * 1000
     end
 
     # Returns the intended execution time for the job.

data/app/models/postburner/job.rb

@@ -3,8 +3,8 @@
 module Postburner
   # Base class for all Postburner background jobs using Single Table Inheritance (STI).
   #
-  # Postburner::Job provides a PostgreSQL-backed job queue system built on Backburner
-  # and Beanstalkd. Every job is stored as an ActiveRecord model, enabling database
+  # Postburner::Job provides a PostgreSQL-backed job queue system built on Beanstalkd.
+  # Every job is stored as an ActiveRecord model, enabling database
   # queries, foreign key relationships, and full audit trails.
   #
   # @abstract Subclass and implement {#perform} to define job behavior
@@ -47,8 +47,7 @@ module Postburner
   # @note Job won't run unless queued_at is set and is prior to execution time
   # @note Subclasses must implement the {#perform} method
   #
-  # @todo Add support for ActiveJob::Arguments serialization
-  # @todo Add `cancelled_at` field to block job execution if present
+  # @todo Add `cancelled_at` field to block job execution if present (See opensource/postburner#16)
   #
   # @attr_reader [Integer] id Primary key
   # @attr_reader [String] type STI type column for job subclass
@@ -79,8 +78,13 @@ module Postburner
     include Execution
     include Statistics
 
+    # Association to schedule execution (if job was created by scheduler)
+    has_one :schedule_execution
+    has_one :schedule, through: :schedule_execution
+
     before_validation :ensure_sid!
     before_destroy :delete!
+    before_attempt :schedule_next_execution
 
     validates :sid, presence: {strict: true}
 
@@ -286,5 +290,24 @@ module Postburner
       self.sid ||= SecureRandom.uuid
     end
 
+    # Before-attempt callback that creates the next schedule execution.
+    #
+    # If this job was created by a schedule, delegates to Schedule to create
+    # the next execution. This provides immediate pickup without waiting for
+    # the scheduler watchdog.
+    #
+    # Only applies to Postburner::Job-based schedules, not ActiveJob schedules.
+    # The scheduler watchdog remains the safety net for all schedules.
+    #
+    # @return [void]
+    #
+    # @api private
+    #
+    def schedule_next_execution
+      return unless schedule_execution.present?
+
+      schedule.create_next_execution!(after: schedule_execution)
+    end
+
   end
 end

data/app/models/postburner/mailer.rb

@@ -2,7 +2,7 @@ module Postburner
   # Send a mailer, tracked.
   #
   # Postburner::Mailer.deliver(UserMailer, :welcome).with(user_id: 1)
-  # Postburner::Mailer.deliver(UserMailer, :welcome).with(user_id: 1).queue! at: Time.zone.now + 1.day
+  # Postburner::Mailer.deliver(UserMailer, :welcome).with(user_id: 1).queue! at: Time.current + 1.day
   # Postburner::Mailer.deliver(UserMailer, :welcome).with(user_id: 1).queue! delay: 5.minutes
   class Mailer < Job
     #queue 'mailers'