workhorse 1.2.17 → 1.2.19

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7c3223be9f4d677fa59d215519c9b96e933e5ab37f3ad96e904cd6dc91b59488
-  data.tar.gz: c30eaa0c15cd8785caf0aab8800de9c8838ff62d9192e1d8d9e8a386659322e1
+  metadata.gz: 4ee79a0aad4b594650de92485a2a0f4d95364042fe1ce8ebd4a2be075827d07d
+  data.tar.gz: a2dcfbdc9e1152513e9fb02613d1f219c3849f85797716e431f0a2fb84fae654
 SHA512:
-  metadata.gz: 9d79e67c853bf5b258ccc2d334de5c746c046b6fca063907cdc3e0a8693e641d12cbefb3ce408f69f697dd1b8f7b0ee4e6ccfcd98ed4b86210786ee53bf23e01
-  data.tar.gz: eca0f05a85ed9a9afaeb0d146b8243b02c3d8bd5cf1229488eacae4437494df1554ce7c2f0cf2f97118b8024d2698d6d054aef3039ab19a253497e4a6caab54f
+  metadata.gz: 99496e59a85fad0b8ca3fe3d9a24b4fd30fd8be3a9e0a236f584dffc42bfda37ba91a5224dfcd29aee88de72ea1df234df286262b5777daf9edb8cfffa0dc2f8
+  data.tar.gz: 13eee5ce97cfc0e66ef4b99a9132590230cc636d202f6e05e9757288dc0ba05e0fd916052bd3638464d0ecb0fafd868004fdc810c4f45f3d0a79350253a17b75

data/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Workhorse Changelog
 
+## 1.2.19 - 2024-05-07
+
+* Fix further compatibility issues with `ruby < 2.5`
+
+  Sitrox reference: #124538.
+
+## 1.2.18 - 2024-05-07
+
+* Fix compatibility with `ruby < 2.5`
+
+  Sitrox reference: #124538.
+
 ## 1.2.17 - 2024-02-21
 
 * Stable release based on previous RC releases.

data/VERSION

@@ -1 +1 @@
-1.2.17
+1.2.19

data/lib/workhorse/performer.rb

@@ -9,41 +9,45 @@ module Workhorse
     end
 
     def perform
-      fail 'Performer can only run once.' if @started
-      @started = true
-      perform!
-    rescue Exception => e
-      Workhorse.on_exception.call(e)
+      begin
+        fail 'Performer can only run once.' if @started
+        @started = true
+        perform!
+      rescue Exception => e
+        Workhorse.on_exception.call(e)
+      end
     end
 
     private
 
     def perform!
-      Thread.current[:workhorse_current_performer] = self
+      begin
+        Thread.current[:workhorse_current_performer] = self
 
-      ActiveRecord::Base.connection_pool.with_connection do
-        if defined?(Rails) && Rails.respond_to?(:application) && Rails.application && Rails.application.respond_to?(:executor)
-          Rails.application.executor.wrap do
+        ActiveRecord::Base.connection_pool.with_connection do
+          if defined?(Rails) && Rails.respond_to?(:application) && Rails.application && Rails.application.respond_to?(:executor)
+            Rails.application.executor.wrap do
+              perform_wrapped
+            end
+          else
             perform_wrapped
           end
-        else
-          perform_wrapped
         end
-      end
-    rescue Exception => e
-      # ---------------------------------------------------------------
-      # Mark job as failed
-      # ---------------------------------------------------------------
-      log %(#{e.message}\n#{e.backtrace.join("\n")}), :error
+      rescue Exception => e
+        # ---------------------------------------------------------------
+        # Mark job as failed
+        # ---------------------------------------------------------------
+        log %(#{e.message}\n#{e.backtrace.join("\n")}), :error
 
-      Workhorse.tx_callback.call do
-        log 'Mark failed', :debug
-        @db_job.mark_failed!(e)
-      end
+        Workhorse.tx_callback.call do
+          log 'Mark failed', :debug
+          @db_job.mark_failed!(e)
+        end
 
-      fail e
-    ensure
-      Thread.current[:workhorse_current_performer] = nil
+        fail e
+      ensure
+        Thread.current[:workhorse_current_performer] = nil
+      end
     end
 
     def perform_wrapped

data/lib/workhorse/poller.rb

@@ -92,10 +92,12 @@ module Workhorse
 
           # Get pids without active process
           orphaned_pids = job_pids.select do |pid|
-            Process.getpgid(pid)
-            false
-          rescue Errno::ESRCH
-            true
+            begin
+              Process.getpgid(pid)
+              false
+            rescue Errno::ESRCH
+              true
+            end
           end
 
           # Reset jobs in state 'locked'
@@ -139,60 +141,62 @@ module Workhorse
     end
 
     def with_global_lock(name: :workhorse, timeout: 2, &_block)
-      if @is_oracle
-        result = Workhorse::DbJob.connection.select_all(
-          "SELECT DBMS_LOCK.REQUEST(#{ORACLE_LOCK_HANDLE}, #{ORACLE_LOCK_MODE}, #{timeout}) FROM DUAL"
-        ).first.values.last
+      begin
+        if @is_oracle
+          result = Workhorse::DbJob.connection.select_all(
+            "SELECT DBMS_LOCK.REQUEST(#{ORACLE_LOCK_HANDLE}, #{ORACLE_LOCK_MODE}, #{timeout}) FROM DUAL"
+          ).first.values.last
 
-        success = result == 0
-      else
-        result = Workhorse::DbJob.connection.select_all(
-          "SELECT GET_LOCK(CONCAT(DATABASE(), '_#{name}'), #{timeout})"
-        ).first.values.last
-        success = result == 1
-      end
+          success = result == 0
+        else
+          result = Workhorse::DbJob.connection.select_all(
+            "SELECT GET_LOCK(CONCAT(DATABASE(), '_#{name}'), #{timeout})"
+          ).first.values.last
+          success = result == 1
+        end
 
-      if success
-        @global_lock_fails = 0
-        @max_global_lock_fails_reached = false
-      else
-        @global_lock_fails += 1
+        if success
+          @global_lock_fails = 0
+          @max_global_lock_fails_reached = false
+        else
+          @global_lock_fails += 1
 
-        unless @max_global_lock_fails_reached
-          worker.log 'Could not obtain global lock, retrying with next poll.', :warn
-        end
+          unless @max_global_lock_fails_reached
+            worker.log 'Could not obtain global lock, retrying with next poll.', :warn
+          end
 
-        if @global_lock_fails > Workhorse.max_global_lock_fails && !@max_global_lock_fails_reached
-          @max_global_lock_fails_reached = true
-
-          worker.log 'Could not obtain global lock, retrying with next poll. ' \
-                     'This will be the last such message for this worker until ' \
-                     'the issue is resolved.', :warn
-
-          message = "Worker reached maximum number of consecutive times (#{Workhorse.max_global_lock_fails}) " \
-                    "where the global lock could no be acquired within the specified timeout (#{timeout}). " \
-                    'A worker that obtained this lock may have crashed without ending the database ' \
-                    'connection properly. On MySQL, use "show processlist;" to see which connection(s) ' \
-                    'is / are holding the lock for a long period of time and consider killing them using ' \
-                    "MySQL's \"kill <Id>\" command. This message will be issued only once per worker " \
-                    'and may only be re-triggered if the error happens again *after* the lock has ' \
-                    'been solved in the meantime.'
-
-          worker.log message
-          exception = StandardError.new(message)
-          Workhorse.on_exception.call(exception)
+          if @global_lock_fails > Workhorse.max_global_lock_fails && !@max_global_lock_fails_reached
+            @max_global_lock_fails_reached = true
+
+            worker.log 'Could not obtain global lock, retrying with next poll. ' \
+              'This will be the last such message for this worker until ' \
+              'the issue is resolved.', :warn
+
+            message = "Worker reached maximum number of consecutive times (#{Workhorse.max_global_lock_fails}) " \
+              "where the global lock could no be acquired within the specified timeout (#{timeout}). " \
+              'A worker that obtained this lock may have crashed without ending the database ' \
+              'connection properly. On MySQL, use "show processlist;" to see which connection(s) ' \
+              'is / are holding the lock for a long period of time and consider killing them using ' \
+              "MySQL's \"kill <Id>\" command. This message will be issued only once per worker " \
+              'and may only be re-triggered if the error happens again *after* the lock has ' \
+              'been solved in the meantime.'
+
+            worker.log message
+            exception = StandardError.new(message)
+            Workhorse.on_exception.call(exception)
+          end
         end
-      end
 
-      return unless success
+        return unless success
 
-      yield
-    ensure
-      if success
-        if @is_oracle
-          Workhorse::DbJob.connection.execute("SELECT DBMS_LOCK.RELEASE(#{ORACLE_LOCK_HANDLE}) FROM DUAL")
-        else
-          Workhorse::DbJob.connection.execute("SELECT RELEASE_LOCK(CONCAT(DATABASE(), '_#{name}'))")
+        yield
+      ensure
+        if success
+          if @is_oracle
+            Workhorse::DbJob.connection.execute("SELECT DBMS_LOCK.RELEASE(#{ORACLE_LOCK_HANDLE}) FROM DUAL")
+          else
+            Workhorse::DbJob.connection.execute("SELECT RELEASE_LOCK(CONCAT(DATABASE(), '_#{name}'))")
+          end
         end
       end
     end

data/lib/workhorse/pool.rb

@@ -34,10 +34,12 @@ module Workhorse
         active_threads.increment
 
         @executor.post do
-          yield
-        ensure
-          active_threads.decrement
-          @on_idle.try(:call)
+          begin
+            yield
+          ensure
+            active_threads.decrement
+            @on_idle.try(:call)
+          end
         end
       end
     end

data/lib/workhorse/worker.rb

@@ -144,19 +144,23 @@ module Workhorse
     end
 
     def perform(db_job_id)
-      mutex.synchronize do
-        assert_state! :running
-        log "Posting job #{db_job_id} to thread pool"
-
-        @pool.post do
-          Workhorse::Performer.new(db_job_id, self).perform
-        rescue Exception => e
-          log %(#{e.message}\n#{e.backtrace.join("\n")}), :error
-          Workhorse.on_exception.call(e)
+      begin
+        mutex.synchronize do
+          assert_state! :running
+          log "Posting job #{db_job_id} to thread pool"
+
+          @pool.post do
+            begin
+              Workhorse::Performer.new(db_job_id, self).perform
+            rescue Exception => e
+              log %(#{e.message}\n#{e.backtrace.join("\n")}), :error
+              Workhorse.on_exception.call(e)
+            end
+          end
         end
+      rescue Exception => e
+        Workhorse.on_exception.call(e)
       end
-    rescue Exception => e
-      Workhorse.on_exception.call(e)
     end
 
     private

data/workhorse.gemspec

@@ -1,14 +1,14 @@
 # -*- encoding: utf-8 -*-
-# stub: workhorse 1.2.17 ruby lib
+# stub: workhorse 1.2.19 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "workhorse".freeze
-  s.version = "1.2.17"
+  s.version = "1.2.19"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib".freeze]
   s.authors = ["Sitrox".freeze]
-  s.date = "2024-02-21"
+  s.date = "2024-05-07"
   s.files = [".github/workflows/ruby.yml".freeze, ".gitignore".freeze, ".releaser_config".freeze, ".rubocop.yml".freeze, "CHANGELOG.md".freeze, "FAQ.md".freeze, "Gemfile".freeze, "LICENSE".freeze, "README.md".freeze, "RUBY_VERSION".freeze, "Rakefile".freeze, "VERSION".freeze, "bin/rubocop".freeze, "lib/active_job/queue_adapters/workhorse_adapter.rb".freeze, "lib/generators/workhorse/install_generator.rb".freeze, "lib/generators/workhorse/templates/bin/workhorse.rb".freeze, "lib/generators/workhorse/templates/config/initializers/workhorse.rb".freeze, "lib/generators/workhorse/templates/create_table_jobs.rb".freeze, "lib/workhorse.rb".freeze, "lib/workhorse/active_job_extension.rb".freeze, "lib/workhorse/daemon.rb".freeze, "lib/workhorse/daemon/shell_handler.rb".freeze, "lib/workhorse/db_job.rb".freeze, "lib/workhorse/enqueuer.rb".freeze, "lib/workhorse/jobs/cleanup_succeeded_jobs.rb".freeze, "lib/workhorse/jobs/detect_stale_jobs_job.rb".freeze, "lib/workhorse/jobs/run_active_job.rb".freeze, "lib/workhorse/jobs/run_rails_op.rb".freeze, "lib/workhorse/performer.rb".freeze, "lib/workhorse/poller.rb".freeze, "lib/workhorse/pool.rb".freeze, "lib/workhorse/scoped_env.rb".freeze, "lib/workhorse/worker.rb".freeze, "test/active_job/queue_adapters/workhorse_adapter_test.rb".freeze, "test/lib/db_schema.rb".freeze, "test/lib/jobs.rb".freeze, "test/lib/test_helper.rb".freeze, "test/workhorse/daemon_test.rb".freeze, "test/workhorse/db_job_test.rb".freeze, "test/workhorse/enqueuer_test.rb".freeze, "test/workhorse/performer_test.rb".freeze, "test/workhorse/poller_test.rb".freeze, "test/workhorse/pool_test.rb".freeze, "test/workhorse/worker_test.rb".freeze, "workhorse.gemspec".freeze]
   s.rubygems_version = "3.4.6".freeze
   s.summary = "Multi-threaded job backend with database queuing for ruby.".freeze

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: workhorse
 version: !ruby/object:Gem::Version
-  version: 1.2.17
+  version: 1.2.19
 platform: ruby
 authors:
 - Sitrox
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-02-21 00:00:00.000000000 Z
+date: 2024-05-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -178,8 +178,8 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: 
-email: 
+description:
+email:
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -229,10 +229,10 @@ files:
 - test/workhorse/pool_test.rb
 - test/workhorse/worker_test.rb
 - workhorse.gemspec
-homepage: 
+homepage:
 licenses: []
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -247,8 +247,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.6
-signing_key: 
+rubygems_version: 3.5.4
+signing_key:
 specification_version: 4
 summary: Multi-threaded job backend with database queuing for ruby.
 test_files: