good_job 3.15.6 → 3.15.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a7428e5e751cae734ce33eecb6a9fc9a4e05bfd3c1774e6d0b6755785b1e33e7
-  data.tar.gz: d753e31cb938aa8fcd5bd1e6a0737022b883b76d734796d4212989f6de987bfe
+  metadata.gz: 49215c024094f667286ca6d15541b8b3d7c046e3f78155a0533e8941e75d2cf9
+  data.tar.gz: 89520dcc01d3a2ffd023077f8e0ba049c9e55f39fc5cfddd1babb70bf7b716a9
 SHA512:
-  metadata.gz: a0bb1f2322318db6be15c5a47ca88b063eb6f1f4a4981a9f78799c531d0f47beb888628e8216f22a7ca51873684cff93062a3d9e7e7da52cac186cca10da841a
-  data.tar.gz: 9e9b91b4e904115033d0daf2aaf7583ba757f318a9f9eb36a005d2ae692e2796abfecea45b2ed4754ecdff218dd86854b6b9ddd8ebb21762a8786621b0d554ae
+  metadata.gz: ed47837b028707b3f6660aba678f0e20a475c2fa56144eedf06edd70c186c1ba08ea8223fd388840c58a6fa2cec75ffb322c386aa93f18ab086ee1ded58e48fb
+  data.tar.gz: 75b05c5abf5cee0a87634371a91c838c86b336c4dcde84d205b6b86748e2e0123b6719d001f78d18d08fd1e8b5f0e96d1a1155e42d7943c9c730697724396bce

data/CHANGELOG.md

@@ -1,5 +1,36 @@
 # Changelog
 
+## [v3.15.8](https://github.com/bensheldon/good_job/tree/v3.15.8) (2023-05-19)
+
+[Full Changelog](https://github.com/bensheldon/good_job/compare/v3.15.7...v3.15.8)
+
+**Closed issues:**
+
+- `active_job.enqueue` Performance Issue [\#939](https://github.com/bensheldon/good_job/issues/939)
+
+**Merged pull requests:**
+
+- Fix hard-coded and duplicated table\_name [\#958](https://github.com/bensheldon/good_job/pull/958) ([gap777](https://github.com/gap777))
+- Bump rubocop from 1.46.0 to 1.50.2 [\#949](https://github.com/bensheldon/good_job/pull/949) ([dependabot[bot]](https://github.com/apps/dependabot))
+- Bump rubocop-rails from 2.18.0 to 2.19.1 [\#947](https://github.com/bensheldon/good_job/pull/947) ([dependabot[bot]](https://github.com/apps/dependabot))
+
+## [v3.15.7](https://github.com/bensheldon/good_job/tree/v3.15.7) (2023-04-30)
+
+[Full Changelog](https://github.com/bensheldon/good_job/compare/v3.15.6...v3.15.7)
+
+**Fixed bugs:**
+
+- Fix Lockable's `SELECT 1 AS one` queries missing `LIMIT 1` [\#946](https://github.com/bensheldon/good_job/pull/946) ([bensheldon](https://github.com/bensheldon))
+
+**Closed issues:**
+
+- Issue with performing jobs that use ActionMailer::Parameterized [\#944](https://github.com/bensheldon/good_job/issues/944)
+- Deprecation warning on the deprecation warning [\#943](https://github.com/bensheldon/good_job/issues/943)
+
+**Merged pull requests:**
+
+- Replace class usage of ActiveSupport::Deprecator with gem-specific instance [\#945](https://github.com/bensheldon/good_job/pull/945) ([bensheldon](https://github.com/bensheldon))
+
 ## [v3.15.6](https://github.com/bensheldon/good_job/tree/v3.15.6) (2023-04-24)
 
 [Full Changelog](https://github.com/bensheldon/good_job/compare/v3.15.5...v3.15.6)

data/app/models/good_job/base_execution.rb

@@ -35,7 +35,7 @@ module GoodJob
       end
 
       def discrete_support?
-        if connection.table_exists?('good_job_executions')
+        if connection.table_exists?(DiscreteExecution.table_name)
           true
         else
           migration_pending_warning!

data/app/models/good_job/base_record.rb

@@ -8,7 +8,7 @@ module GoodJob
     self.abstract_class = true
 
     def self.migration_pending_warning!
-      ActiveSupport::Deprecation.warn(<<~DEPRECATION)
+      GoodJob.deprecator.warn(<<~DEPRECATION)
         GoodJob has pending database migrations. To create the migration files, run:
             rails generate good_job:update
         To apply the migration files, run:

data/app/models/good_job/execution.rb

@@ -212,7 +212,7 @@ module GoodJob
     # Construct arguments for GoodJob::Execution from an ActiveJob instance.
     def self.enqueue_args(active_job, overrides = {})
       if active_job.priority && GoodJob.configuration.smaller_number_is_higher_priority.nil?
-        ActiveSupport::Deprecation.warn(<<~DEPRECATION)
+        GoodJob.deprecator.warn(<<~DEPRECATION)
           The next major version of GoodJob (v4.0) will change job `priority` to give smaller numbers higher priority (default: `0`), in accordance with Active Job's definition of priority.
             To opt-in to this behavior now, set `config.good_job.smaller_number_is_higher_priority = true` in your GoodJob initializer or application.rb.
             To not opt-in yet, but silence this deprecation warning, set `config.good_job.smaller_number_is_higher_priority = false`.

data/app/models/good_job/lockable.rb

@@ -324,6 +324,7 @@ module GoodJob
           AND pg_locks.objsubid = 1
           AND pg_locks.classid = ('x' || substr(md5($1::text), 1, 16))::bit(32)::int
           AND pg_locks.objid = (('x' || substr(md5($2::text), 1, 16))::bit(64) << 32)::bit(32)::int
+        LIMIT 1
       SQL
       binds = [
         ActiveRecord::Relation::QueryAttribute.new('key', key, ActiveRecord::Type::String.new),
@@ -351,6 +352,7 @@ module GoodJob
           AND pg_locks.classid = ('x' || substr(md5($1::text), 1, 16))::bit(32)::int
           AND pg_locks.objid = (('x' || substr(md5($2::text), 1, 16))::bit(64) << 32)::bit(32)::int
           AND pg_locks.pid = pg_backend_pid()
+        LIMIT 1
       SQL
       binds = [
         ActiveRecord::Relation::QueryAttribute.new('key', key, ActiveRecord::Type::String.new),

data/lib/good_job/configuration.rb

@@ -250,12 +250,12 @@ module GoodJob
       if rails_config.key?(:cleanup_interval_jobs)
         value = rails_config[:cleanup_interval_jobs]
         if value.nil?
-          ActiveSupport::Deprecation.warn(
+          GoodJob.deprecator.warn(
             %(Setting `config.good_job.cleanup_interval_jobs` to `nil` will no longer disable count-based cleanups in GoodJob v4. Set to `false` to disable, or `-1` to run every time.)
           )
           value = false
         elsif value == 0 # rubocop:disable Style/NumericPredicate
-          ActiveSupport::Deprecation.warn(
+          GoodJob.deprecator.warn(
             %(Setting `config.good_job.cleanup_interval_jobs` to `0` will disable count-based cleanups in GoodJob v4. Set to `false` to disable, or `-1` to run every time.)
           )
           value = -1
@@ -263,7 +263,7 @@ module GoodJob
       elsif env.key?('GOOD_JOB_CLEANUP_INTERVAL_JOBS')
         value = env['GOOD_JOB_CLEANUP_INTERVAL_JOBS']
         if value.blank?
-          ActiveSupport::Deprecation.warn(
+          GoodJob.deprecator.warn(
             %(Setting `GOOD_JOB_CLEANUP_INTERVAL_JOBS` to `""` will no longer disable count-based cleanups in GoodJob v4. Set to `0` to disable, or `-1` to run every time.)
           )
           value = false
@@ -285,12 +285,12 @@ module GoodJob
         value = rails_config[:cleanup_interval_seconds]
 
         if value.nil?
-          ActiveSupport::Deprecation.warn(
+          GoodJob.deprecator.warn(
             %(Setting `config.good_job.cleanup_interval_seconds` to `nil` will no longer disable time-based cleanups in GoodJob v4. Set to `false` to disable, or `-1` to run every time.)
           )
           value = false
         elsif value == 0 # rubocop:disable Style/NumericPredicate
-          ActiveSupport::Deprecation.warn(
+          GoodJob.deprecator.warn(
             %(Setting `config.good_job.cleanup_interval_seconds` to `0` will disable time-based cleanups in GoodJob v4. Set to `false` to disable, or `-1` to run every time.)
           )
           value = -1
@@ -298,7 +298,7 @@ module GoodJob
       elsif env.key?('GOOD_JOB_CLEANUP_INTERVAL_SECONDS')
         value = env['GOOD_JOB_CLEANUP_INTERVAL_SECONDS']
         if value.blank?
-          ActiveSupport::Deprecation.warn(
+          GoodJob.deprecator.warn(
             %(Setting `GOOD_JOB_CLEANUP_INTERVAL_SECONDS` to `""` will no longer disable time-based cleanups in GoodJob v4. Set to `0` to disable, or `-1` to run every time.)
           )
           value = false

data/lib/good_job/version.rb

@@ -1,5 +1,8 @@
 # frozen_string_literal: true
 module GoodJob
   # GoodJob gem version.
-  VERSION = '3.15.6'
+  VERSION = '3.15.8'
+
+  # GoodJob version as Gem::Version object
+  GEM_VERSION = Gem::Version.new(VERSION)
 end

data/lib/good_job.rb

@@ -223,5 +223,14 @@ module GoodJob
     end
   end
 
+  # Deprecator for providing deprecation warnings.
+  # @return [ActiveSupport::Deprecation]
+  def self.deprecator
+    @_deprecator ||= begin
+      next_major_version = GEM_VERSION.segments[0] + 1
+      ActiveSupport::Deprecation.new("#{next_major_version}.0", "GoodJob")
+    end
+  end
+
   ActiveSupport.run_load_hooks(:good_job, self)
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: good_job
 version: !ruby/object:Gem::Version
-  version: 3.15.6
+  version: 3.15.8
 platform: ruby
 authors:
 - Ben Sheldon
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-04-24 00:00:00.000000000 Z
+date: 2023-05-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activejob