good_job 2.15.1 → 2.16.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bef9bcf112b4c04ebbc2ebb05a6253bfafcf8d6b077ac177e2da7dc51beb2530
-  data.tar.gz: c5379f8a78edf855415b67f0879e65ad469eebf73c535576d572a0069ae7feff
+  metadata.gz: 6f37fb872be59c7ead625a1ff54ad50d39206209c87bf11f8a88bd7f2f484676
+  data.tar.gz: 50c80eebdd8654894ad26ed052fa0ddfedac8dd30e100e260541d0b1d21d6951
 SHA512:
-  metadata.gz: 6f668a1496b0a5ef3b9c0a8d126fe4b6709bcabd1cb96466fe4c8530620dffeed9ac2c10ca103773bf2a7196441d8bc8981893a4f01c91bd5cddf318d08fbf05
-  data.tar.gz: 26f017289703d8017dd69183b425a602d1501e835b11c4552ecfca789ed43ee192d56aaed2c204ecc06343ac07908b19ad56b23d978a0cae5869a10236e75dbd
+  metadata.gz: c6aff64526dcc2989f14e6a7b4fe1a5be5a13df166a13c2055f10eada589f0f8d9586be7a98be94e4e16f81c55524c260cb28b52771a8f8a62064553fa426b5e
+  data.tar.gz: 4f64c31ecf4ff9149d35c0611b1d04158ec7ee1f92cefcbce831f1c442d551d7f406a818d4a111b7c4e848d2ecf15f66dd0968d6224f5d51c2a65394075b3237

data/CHANGELOG.md

@@ -1,5 +1,38 @@
 # Changelog
 
+## [v2.16.1](https://github.com/bensheldon/good_job/tree/v2.16.1) (2022-06-18)
+
+[Full Changelog](https://github.com/bensheldon/good_job/compare/v2.16.0...v2.16.1)
+
+**Merged pull requests:**
+
+- Fix `:inline` mode with future behavior to run unscheduled jobs immediately [\#620](https://github.com/bensheldon/good_job/pull/620) ([bensheldon](https://github.com/bensheldon))
+- Version check `Zeitwerk::Loader.new(warn_on_extra_files: false)` flag [\#619](https://github.com/bensheldon/good_job/pull/619) ([bensheldon](https://github.com/bensheldon))
+
+## [v2.16.0](https://github.com/bensheldon/good_job/tree/v2.16.0) (2022-06-17)
+
+[Full Changelog](https://github.com/bensheldon/good_job/compare/v2.15.2...v2.16.0)
+
+**Closed issues:**
+
+- Upgrading zeitwerk to 2.6.0 causes a warning related to good\_job [\#616](https://github.com/bensheldon/good_job/issues/616)
+
+**Merged pull requests:**
+
+- Allow inline executor to respect scheduled jobs; deprecate old behavior. Add `GoodJob.perform_inline` [\#615](https://github.com/bensheldon/good_job/pull/615) ([bensheldon](https://github.com/bensheldon))
+
+## [v2.15.2](https://github.com/bensheldon/good_job/tree/v2.15.2) (2022-06-17)
+
+[Full Changelog](https://github.com/bensheldon/good_job/compare/v2.15.1...v2.15.2)
+
+**Closed issues:**
+
+- ActiveRecord::StatementInvalid PG::ProgramLimitExceeded:  ERROR: index row size 3296 exceeds btree version 4 maximum 2704 for index [\#612](https://github.com/bensheldon/good_job/issues/612)
+
+**Merged pull requests:**
+
+- Zeitwerk ignore `lib/active_job` [\#617](https://github.com/bensheldon/good_job/pull/617) ([bensheldon](https://github.com/bensheldon))
+
 ## [v2.15.1](https://github.com/bensheldon/good_job/tree/v2.15.1) (2022-05-24)
 
 [Full Changelog](https://github.com/bensheldon/good_job/compare/v2.15.0...v2.15.1)
@@ -22,6 +55,7 @@
 
 **Implemented enhancements:**
 
+- Remove ability to destroy individual Executions from Dashboard; rename "Toggle" to "Inspect" everywhere [\#601](https://github.com/bensheldon/good_job/pull/601) ([bensheldon](https://github.com/bensheldon))
 - Adds the ability to delete jobs on the dashboard; add `cleanup_discarded_jobs` option to retain discarded jobs during cleanup [\#597](https://github.com/bensheldon/good_job/pull/597) ([TAGraves](https://github.com/TAGraves))
 - Dashboard: show more details about jobs [\#575](https://github.com/bensheldon/good_job/pull/575) ([bkeepers](https://github.com/bkeepers))
 
@@ -31,7 +65,6 @@
 
 **Merged pull requests:**
 
-- Remove ability to destroy individual Executions from Dashboard; rename "Toggle" to "Inspect" everywhere [\#601](https://github.com/bensheldon/good_job/pull/601) ([bensheldon](https://github.com/bensheldon))
 - Disable ActiveRecord Connection Reaper in test [\#600](https://github.com/bensheldon/good_job/pull/600) ([bensheldon](https://github.com/bensheldon))
 - Update README dashboard screenshot [\#599](https://github.com/bensheldon/good_job/pull/599) ([aried3r](https://github.com/aried3r))
 

data/README.md

@@ -59,6 +59,7 @@ For more of the story of GoodJob, read the [introductory blog post](https://isla
     - [Execute jobs async / in-process](#execute-jobs-async--in-process)
     - [Migrate to GoodJob from a different ActiveJob backend](#migrate-to-goodjob-from-a-different-activejob-backend)
     - [Monitor and preserve worked jobs](#monitor-and-preserve-worked-jobs)
+    - [Write tests](#write-tests)
     - [PgBouncer compatibility](#pgbouncer-compatibility)
     - [CLI HTTP health check probes](#cli-http-health-check-probes)
 - [Contribute](#contribute)
@@ -276,6 +277,7 @@ Available configuration options are:
 - `cleanup_preserved_jobs_before_seconds_ago` (integer) number of seconds to preserve jobs when using the `$ good_job cleanup_preserved_jobs` CLI command or calling `GoodJob.cleanup_preserved_jobs`. Defaults to `86400` (1 day). Can also be set with  the environment variable `GOOD_JOB_CLEANUP_PRESERVED_JOBS_BEFORE_SECONDS_AGO`.  _This configuration is only used when {GoodJob.preserve_job_records} is `true`._
 - `cleanup_interval_jobs` (integer) Number of jobs a Scheduler will execute before cleaning up preserved jobs. Defaults to `nil`. Can also be set with  the environment variable `GOOD_JOB_CLEANUP_INTERVAL_JOBS`.
 - `cleanup_interval_seconds` (integer) Number of seconds a Scheduler will wait before cleaning up preserved jobs. Defaults to `nil`. Can also be set with  the environment variable `GOOD_JOB_CLEANUP_INTERVAL_SECONDS`.
+- `inline_execution_respects_schedule` (boolean) Opt-in to future behavior of inline execution respecting scheduled jobs. Defaults to `false`.
 - `logger` ([Rails Logger](https://api.rubyonrails.org/classes/ActiveSupport/Logger.html)) lets you set a custom logger for GoodJob. It should be an instance of a Rails `Logger` (Default: `Rails.logger`).
 - `preserve_job_records` (boolean) keeps job records in your database even after jobs are completed. (Default: `false`)
 - `retry_on_unhandled_error` (boolean) causes jobs to be re-queued and retried if they raise an instance of `StandardError`. Be advised this may lead to jobs being repeated infinitely ([see below for more on retries](#retries)). Instances of `Exception`, like SIGINT, will *always* be retried, regardless of this attribute’s value. (Default: `true`)
@@ -878,6 +880,23 @@ It is also necessary to destroy these preserved jobs from the database after a c
     bundle exec good_job cleanup_preserved_jobs --before-seconds-ago=86400
     ```
 
+### Write tests
+
+By default, GoodJob uses its inline adapter in the test environment; the inline adapter is designed for the test environment. When enquing a job with GoodJob's inline adapter, the job will be executed immediately on the current thread; unhandled exceptions will be raised.
+
+In GoodJob 2.0, the inline adapter will execute future scheduled jobs immediately. In the next major release, GoodJob 3.0, the inline adapter will not execute future scheduled jobs and instead enqueue them in the database.
+
+To opt into this behavior immediately set: `config.good_job.inline_execution_respects_schedule = true`
+
+To perform jobs inline at any time, use `GoodJob.perform_inline`. For example, using time helpers within an integration test:
+
+```ruby
+MyJob.set(wait: 10.minutes).perform_later
+travel_to(15.minutes.from_now) { GoodJob.perform_inline }
+```
+
+_Note: Rails `travel`/`travel_to` time helpers do not have millisecond precision, so you must leave at least 1 second between the schedule and time traveling for the job to be executed. This [behavior may change in Rails 7.1](https://github.com/rails/rails/pull/44088)._
+
 ### PgBouncer compatibility
 
 GoodJob is not compatible with PgBouncer in _transaction_ mode, but is compatible with PgBouncer's _connection_ mode. GoodJob uses connection-based advisory locks and LISTEN/NOTIFY, both of which require full database connections.

data/lib/good_job/adapter.rb

@@ -62,19 +62,48 @@ module GoodJob
     # @param timestamp [Integer, nil] the epoch time to perform the job
     # @return [GoodJob::Execution]
     def enqueue_at(active_job, timestamp)
+      scheduled_at = timestamp ? Time.zone.at(timestamp) : nil
+
+      if execute_inline?
+        future_scheduled = scheduled_at && scheduled_at > Time.current
+        will_execute_inline = !future_scheduled || (future_scheduled && !@configuration.inline_execution_respects_schedule?)
+      end
+
       execution = GoodJob::Execution.enqueue(
         active_job,
-        scheduled_at: timestamp ? Time.zone.at(timestamp) : nil,
-        create_with_advisory_lock: execute_inline?
+        scheduled_at: scheduled_at,
+        create_with_advisory_lock: will_execute_inline
       )
 
-      if execute_inline?
+      if will_execute_inline
+        if future_scheduled && !@configuration.inline_execution_respects_schedule?
+          ActiveSupport::Deprecation.warn(<<~DEPRECATION)
+            In the next major release, GoodJob will not *inline* execute
+            future-scheduled jobs.
+
+            To opt into this behavior immediately set:
+            `config.good_job.inline_execution_respects_schedule = true`
+
+            To perform jobs inline at any time, use `GoodJob.perform_inline`.
+
+            For example, using time helpers within an integration test:
+
+            ```
+            MyJob.set(wait: 10.minutes).perform_later
+            travel_to(15.minutes.from_now) { GoodJob.perform_inline }
+            ```
+
+            Note: Rails `travel`/`travel_to` time helpers do not have millisecond
+            precision, so you must leave at least 1 second between the schedule
+            and time traveling for the job to be executed.
+          DEPRECATION
+        end
+
         begin
           result = execution.perform
         ensure
           execution.advisory_unlock
         end
-
         raise result.unhandled_error if result.unhandled_error
       else
         job_state = { queue_name: execution.queue_name }

data/lib/good_job/configuration.rb

@@ -119,6 +119,10 @@ module GoodJob
       end
     end
 
+    def inline_execution_respects_schedule?
+      !!rails_config[:inline_execution_respects_schedule]
+    end
+
     # The maximum number of future-scheduled jobs to store in memory.
     # Storing future-scheduled jobs in memory reduces execution latency
     # at the cost of increased memory usage. 10,000 stored jobs = ~20MB.

data/lib/good_job/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 module GoodJob
   # GoodJob gem version.
-  VERSION = '2.15.1'
+  VERSION = '2.16.1'
 end

data/lib/good_job.rb

@@ -4,11 +4,12 @@ require "active_job"
 require "active_job/queue_adapters"
 
 require "zeitwerk"
-Zeitwerk::Loader.for_gem.tap do |loader|
+zeitwerk_options = Gem::Version.new(Zeitwerk::VERSION) >= Gem::Version.new("2.6.0") ? { warn_on_extra_files: false } : {}
+Zeitwerk::Loader.for_gem(**zeitwerk_options).tap do |loader|
   loader.inflector.inflect({
                              "cli" => "CLI",
                            })
-  loader.ignore(File.join(File.dirname(__FILE__), "generators"))
+  loader.ignore("#{__dir__}/generators")
   loader.setup
 end
 
@@ -151,6 +152,20 @@ module GoodJob
     end
   end
 
+  # Perform all queued jobs in the current thread.
+  # This is primarily intended for usage in a test environment.
+  # Unhandled job errors will be raised.
+  # @param queue_string [String] Queues to execute jobs from
+  # @return [void]
+  def self.perform_inline(queue_string = "*")
+    job_performer = JobPerformer.new(queue_string)
+    loop do
+      result = job_performer.next
+      break unless result
+      raise result.unhandled_error if result.unhandled_error
+    end
+  end
+
   def self._executables
     [].concat(
       CronManager.instances,

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: good_job
 version: !ruby/object:Gem::Version
-  version: 2.15.1
+  version: 2.16.1
 platform: ruby
 authors:
 - Ben Sheldon
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-05-24 00:00:00.000000000 Z
+date: 2022-06-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activejob