active_job_store 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a9fd49a85e50473450a09a741acaf01d5e6565116cb6f945f314bd6e4dd74f6d
-  data.tar.gz: d5655ec674b4a86d99b267c8a851e2a8d1cd687e188c58c8a292bf5120cffc3b
+  metadata.gz: 0ef6d562f0f4a15a496ba433ec3c540e141936afb70c25c49d733989c5244662
+  data.tar.gz: bb6790c404ed7aecfc16a54838a01a25d366221e23898cb9bd52e73b2311de9e
 SHA512:
-  metadata.gz: f9e1a166e181d32afb0265ac062b954930b8d748af36b21e837d2ee6fe28a7d57edc2f9ad590339e49fefbc3256e6b43317d19ab4e9ff1de36e7f2aa67d0dac4
-  data.tar.gz: a407a4446dec032e4bf6f7b5f55536dadd703c0d1d1a75d61cdf37ace3388e09c836c520ffde6c0b31043867bd92c82ad04c57b54d7398ca126baa080715e0a5
+  metadata.gz: 5a88f03f2af219714771a7d9dcf320e72d99753f4327ce7f1c699ef8f0132bf62d124872e83ae2be589d88afba4f06aaecdc9506fc13cfe87b36e0777b46c4a3
+  data.tar.gz: b2544a3499f8a0d3a18b34fee58e171ef37cc3402890e1790746c75b97a6f22c236582185769738301aca986afa05e44fa27645a76a75f95480368dbee376aaf

data/README.md

@@ -11,6 +11,8 @@ It can be useful to:
 - query historical data about job executions / extract job's statistical data;
 - improve jobs' logging capabilities.
 
+By default gem's internal errors are sent to stderr without compromising the job's perform.
+
 Please ⭐ if you like it.
 
 ## Installation
@@ -28,6 +30,7 @@ attr_accessor on the jobs:
 
 Instance methods on the jobs:
 - `active_job_store_format_result(result) => result2`: to format / manipulate / serialize the job result
+- `active_job_store_internal_error(context, exception)`: handler for internal errors
 - `active_job_store_record => store record`: returns the store's record
 - `save_job_custom_data(custom_data = nil)`: to persist custom data while the job is performing
 
@@ -87,7 +90,7 @@ puts ::ActiveJobStore::Record.order(id: :desc).pluck(:created_at, :job_class, :a
 # 2022-11-09 21:09:50 UTC, SomeJob, Some test, completed, 2022-11-09 21:09:50 UTC
 ```
 
-Query information from a job (even when it's performing):
+Query information from a job (even while performing):
 
 ```rb
 job = SomeJob.perform_later 123
@@ -102,9 +105,9 @@ job.active_job_store_record.reload.custom_data
 # => {"progress"=>1.0}
 ```
 
-## Features' details
+## Features' examples
 
-To store the custom data (ex. a progress value):
+To persist some custom data during the perform (ex. a progress value):
 
 ```rb
 class AnotherJob < ApplicationJob
@@ -125,7 +128,7 @@ AnotherJob.perform_later(456)
 AnotherJob.job_executions.last.custom_data['progress'] # 1.0 (at the end)
 ```
 
-To manipulate the custom data, there is the `active_job_store_custom_data` accessor:
+To manipulate the custom data persisted only at the end:
 
 ```rb
 class AnotherJob < ApplicationJob
@@ -148,7 +151,7 @@ AnotherJob.job_executions.last.custom_data
 # => [{"time"=>"2022-11-09T21:20:57.580Z", "message"=>"SomeJob step 1"}, {"time"=>"2022-11-09T21:20:58.581Z", "message"=>"SomeJob step 2"}]
 ```
 
-To process the result before storing it (ex. for serialization), override `active_job_store_format_result`:
+To process the job's result before storing it (ex. for serialization):
 
 ```rb
 class AnotherJob < ApplicationJob
@@ -169,6 +172,21 @@ AnotherJob.job_executions.last.result
 # => 84
 ```
 
+To raise an exception also when there is a gem's internal error:
+
+```rb
+class AnotherJob < ApplicationJob
+  include ActiveJobStore
+
+  # ...
+
+  def active_job_store_internal_error(context, exception)
+    raise exception
+    # Or simply monitor these errors using services like Sentry/Honeybadger/etc.
+  end
+end
+```
+
 ## Do you like it? Star it!
 
 If you use this component just star it. A developer is more motivated to improve a project when there is some interest.

data/lib/active_job_store/store.rb

@@ -8,9 +8,11 @@ module ActiveJobStore
 
     def around_enqueue(job)
       prepare_record_on_enqueue(job)
-      record.lock! # NOTE: needed to avoid update conflicts with perform when setting the state to enqueued
-      yield
+      lock_record!
+      result = yield
       job_enqueued!
+      job.active_job_store_internal_error('ActiveJobStore::Store around_enqueue', internal_error) if internal_error
+      result
     end
 
     def around_perform(job)
@@ -19,51 +21,71 @@ module ActiveJobStore
       result = yield
       formatted_result = job.active_job_store_format_result(result)
       job_competed!(custom_data: job.active_job_store_custom_data, result: formatted_result)
+      job.active_job_store_internal_error('ActiveJobStore::Store around_perform', internal_error) if internal_error
+      result
     rescue StandardError => e
       job_failed!(exception: e, custom_data: job.active_job_store_custom_data)
       raise
     end
 
     def update_job_custom_data(custom_data)
-      record.update!(custom_data: custom_data)
+      wrap_errors do
+        record.update!(custom_data: custom_data)
+      end
     end
 
     private
 
+    attr_accessor :internal_error
+
     def job_competed!(result:, custom_data:)
-      record.update!(state: :completed, completed_at: Time.current, result: result, custom_data: custom_data)
-      record
+      wrap_errors do
+        record.update!(state: :completed, completed_at: Time.current, result: result, custom_data: custom_data)
+      end
     end
 
     def job_enqueued!
-      record.update!(state: :enqueued, enqueued_at: Time.current)
-      record
+      wrap_errors do
+        record.update!(state: :enqueued, enqueued_at: Time.current)
+      end
     end
 
     def job_failed!(exception:, custom_data:)
-      record.update!(state: :error, exception: exception.inspect, custom_data: custom_data)
-      record
+      wrap_errors do
+        record.update!(state: :error, exception: exception.inspect, custom_data: custom_data)
+      end
     end
 
     def job_started!
-      record.update!(state: :started, started_at: Time.current)
-      record
+      wrap_errors do
+        record.update!(state: :started, started_at: Time.current)
+      end
+    end
+
+    def lock_record!
+      wrap_errors do
+        record.lock! # NOTE: needed to avoid update conflicts with perform when setting the state to enqueued
+      end
     end
 
     def prepare_record_on_enqueue(job)
-      @record = ::ActiveJobStore::Record.find_or_create_by!(record_reference(job)) do |record|
-        record.arguments = job.arguments
-        record.details = DETAILS_ATTRS.zip(DETAILS_ATTRS.map { job.send(_1) }).to_h
-        record.state = :initialized
+      wrap_errors do
+        @record = ::ActiveJobStore::Record.find_or_create_by!(record_reference(job)) do |record|
+          record.arguments = job.arguments
+          record.details = DETAILS_ATTRS.zip(DETAILS_ATTRS.map { job.send(_1) }).to_h
+          record.state = :initialized
+        end
       end
     end
 
     def prepare_record_on_perform(job)
-      @record = ::ActiveJobStore::Record.find_or_initialize_by(record_reference(job)) do |record|
-        record.arguments = job.arguments
+      wrap_errors do
+        @record = ::ActiveJobStore::Record.find_or_initialize_by(record_reference(job)) do |record|
+          record.arguments = job.arguments
+        end
+        record.details = DETAILS_ATTRS.zip(DETAILS_ATTRS.map { job.send(_1) }).to_h
+        record
       end
-      record.details = DETAILS_ATTRS.zip(DETAILS_ATTRS.map { job.send(_1) }).to_h
-      record
     end
 
     def record_reference(job)
@@ -72,5 +94,13 @@ module ActiveJobStore
         job_class: job.class.to_s
       }
     end
+
+    def wrap_errors
+      return if internal_error
+
+      yield
+    rescue StandardError => e
+      self.internal_error = e
+    end
   end
 end

data/lib/active_job_store/version.rb

@@ -2,6 +2,6 @@
 
 # :nocov:
 module ActiveJobStore
-  VERSION = '0.3.0'
+  VERSION = '0.4.0'
 end
 # :nocov:

data/lib/active_job_store.rb

@@ -31,6 +31,13 @@ module ActiveJobStore
     store.record
   end
 
+  # Internal errors handler method
+  #
+  # @param exception [exception] The internal exception
+  def active_job_store_internal_error(context, exception)
+    warn("#{context}: #{exception}")
+  end
+
   module ClassMethods
     # Query the list of job executions for the current job class
     #

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: active_job_store
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Mattia Roccoberton
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-11-15 00:00:00.000000000 Z
+date: 2022-11-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activejob
@@ -60,7 +60,7 @@ metadata:
   source_code_uri: https://github.com/blocknotes/active_job_store
   changelog_uri: https://github.com/blocknotes/active_job_store/blob/master/CHANGELOG.md
   rubygems_mfa_required: 'true'
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -76,7 +76,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.1.6
-signing_key:
+signing_key: 
 specification_version: 4
 summary: Persist jobs information on DB
 test_files: []