sidekiq_publisher 1.7.0.rc1 → 2.0.0.rc0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7b46548489774867e03fcbd1c4359909cc12d4baf7a92091d06bf45bba3d3120
-  data.tar.gz: c8f0fcd1ec8127f62e880a6282baeeb8b851583d06db25d0ac2e3d85287e1e32
+  metadata.gz: 1bef2eda10ec3e9fc09d95fe0af551b408331b8d3a42ff35d6c0f1012ea744e2
+  data.tar.gz: 317a3f1d450d71e0504fc351b33792002f83a0c9d36e2e2f6cf1c985d546c62b
 SHA512:
-  metadata.gz: 36d2cfa5d35ef29acde6294318d34fbbccf2960d2bd5304ec35bdf0b1c76d4abf6f4130daf0cd37046f101242e236c4855e2a777a7ca6cddb62ca8ae0fad07ea
-  data.tar.gz: 1ac0af71177ed8b6188f0c845e2bb832c73f911326cb8b078f64ce506ce80bc5d6078b0a5a6c9ae0c29839a996f3d92f43f417e2efd38320b615b57c73ac8dbf
+  metadata.gz: c6dc3e3765c0b2e819087aa249add2fb2baf7b625394def4702e7340d210aeb6e2f51be9d529d19be4731742ec66f7a940867f8f28b1dd09112abafc7a620929
+  data.tar.gz: '09598e9b91cd23c47f8aa583155acfb16d54b01a534ef727463f77692999776f29db0df7dfa826b961688f9e20ed6beda568e5c6bd8abb76148717411604f2ce'

data/Appraisals

@@ -1,10 +1,5 @@
 # frozen_string_literal: true
 
-appraise "rails-5.1" do
-  gem "activejob", "~> 5.1.6"
-  gem "activesupport", "~> 5.1.6"
-end
-
 appraise "rails-5.2-sidekiq-5.0" do
   gem "activejob", "~> 5.2.0"
   gem "activesupport", "~> 5.2.0"
@@ -52,3 +47,21 @@ appraise "rails-6.0-sidekiq-6.1" do
   gem "activesupport", "~> 6.0.0"
   gem "sidekiq", "~> 6.1.0"
 end
+
+appraise "rails-6.1-sidekiq-5.2" do
+  gem "activejob", "~> 6.1.0"
+  gem "activesupport", "~> 6.1.0"
+  gem "sidekiq", "~> 5.2.0"
+end
+
+appraise "rails-6.1-sidekiq-6.0" do
+  gem "activejob", "~> 6.1.0"
+  gem "activesupport", "~> 6.1.0"
+  gem "sidekiq", "~> 6.0.1"
+end
+
+appraise "rails-6.1-sidekiq-6.1" do
+  gem "activejob", "~> 6.1.0"
+  gem "activesupport", "~> 6.1.0"
+  gem "sidekiq", "~> 6.1.0"
+end

data/CHANGELOG.md

@@ -1,6 +1,20 @@
 # sidekiq_publisher
 
-## v1.7.0 (unreleased)
+## v2.0.0 (unreleased)
+- Transition from defining a Railtie to becoming a Rails Engine
+  [(#44)](https://github.com/ezcater/sidekiq_publisher/pull/44). This change was
+  made to better support Ruby 3.0 and Rails 6.1. The change is *not* expected to
+  be breaking but it does represent a major change in the gem's implementation
+  change and that is the reason for the major version bump.
+- Require Ruby 2.6 or later.
+
+## v1.8.0
+- Extend support to Rails 6.1.
+
+## v1.7.1
+- Gracefully handle database connection errors in ReportUnpublishedCount by attempting to reconnect.
+
+## v1.7.0
 - Add instrumentation using `ActiveSupport::Notifications`.
 - Reimplement `metrics_reporter` and `exception_reporter` support using
   `ActiveSupport::Subscriber`.

data/Dockerfile

@@ -1,4 +1,4 @@
-FROM ezcater-production.jfrog.io/ruby:f08726283c
+FROM ezcater-production.jfrog.io/ruby:90790156c4
 RUN mkdir /usr/src/gem
 WORKDIR /usr/src/gem
 ADD . /usr/src/gem

data/README.md

@@ -136,6 +136,21 @@ class MyJob < ApplicationJob
 end
 ```
 
+#### ActiveJob Exception Reporting
+
+Many exception monitoring service (e.g. Sentry, Airbrake, Honeybadger, etc) already provide basic integration support for `Sidekiq`. 
+These integration should also work with `SidekiqPublisher`. 
+However, you may need to explicitly include 
+`ActiveJob::QueueAdapters::SidekiqPublisherAdapter` as a compatible adapter for this to work properly.
+
+Alternatively, you can manually report the exception:
+
+ ```ruby
+retry_on SomeError, attempts: 10 do |_job, exception|
+  Raven.capture_exception(exception, extra: { custom: :foo }) # Reporting using the Sentry gem 
+end
+```
+
 ### SidekiqPublisher::Worker
 
 Sidekiq workers are usually defined by including `Sidekiq::Worker` in a class.

data/{lib → app/models}/sidekiq_publisher/job.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require "active_record"
-
 module SidekiqPublisher
   class Job < ActiveRecord::Base
     self.table_name = "sidekiq_publisher_jobs"

data/docker-compose.yml

@@ -7,6 +7,7 @@ x-environment: &default-environment
   BUNDLE_IGNORE_CONFIG: 1
   BUNDLE_DISABLE_SHARED_GEMS: "true"
   PGUSER: postgres
+  PGPASSWORD: postgres
   PGHOST: sidekiq-publisher-postgres
   PGPORT: 5432
   PGDATABASE: sidekiq_publisher_test
@@ -28,9 +29,10 @@ services:
     image: redis:5.0.9-alpine
   sidekiq-publisher-postgres:
     container_name: sidekiq-publisher-postgres_1
-    image: postgres:11.6
+    image: postgres:12.6
     environment:
       POSTGRES_USER: postgres
+      POSTGRES_PASSWORD: postgres
       POSTGRES_DB: sidekiq_publisher_test
   sidekiq-publisher-console:
     <<: *default-service

data/gemfiles/{rails_5.1.gemfile → rails_6.1_sidekiq_5.2.gemfile}

@@ -2,7 +2,8 @@
 
 source "https://rubygems.org"
 
-gem "activejob", "~> 5.1.6"
-gem "activesupport", "~> 5.1.6"
+gem "activejob", "~> 6.1.0"
+gem "activesupport", "~> 6.1.0"
+gem "sidekiq", "~> 5.2.0"
 
 gemspec path: "../"

data/gemfiles/rails_6.1_sidekiq_6.0.gemfile

@@ -0,0 +1,9 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "activejob", "~> 6.1.0"
+gem "activesupport", "~> 6.1.0"
+gem "sidekiq", "~> 6.0.1"
+
+gemspec path: "../"

data/gemfiles/rails_6.1_sidekiq_6.1.gemfile

@@ -0,0 +1,9 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "activejob", "~> 6.1.0"
+gem "activesupport", "~> 6.1.0"
+gem "sidekiq", "~> 6.1.0"
+
+gemspec path: "../"

data/lib/active_job/queue_adapters/sidekiq_publisher_adapter.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require "active_job/queue_adapters/sidekiq_adapter"
+require "active_support/core_ext/object/blank"
 
 module ActiveJob
   module QueueAdapters

data/lib/sidekiq_publisher.rb

@@ -1,16 +1,16 @@
 # frozen_string_literal: true
 
 require "active_support"
+require "active_support/core_ext/numeric/time"
 require "sidekiq_publisher/version"
 require "sidekiq_publisher/instrumenter"
 require "sidekiq_publisher/metrics_reporter"
 require "sidekiq_publisher/exception_reporter"
 require "sidekiq_publisher/report_unpublished_count"
-require "sidekiq_publisher/job"
 require "sidekiq_publisher/worker"
 require "sidekiq_publisher/publisher"
 require "sidekiq_publisher/runner"
-require "sidekiq_publisher/railtie" if defined?(Rails)
+require "sidekiq_publisher/engine" if defined?(Rails)
 
 module SidekiqPublisher
   DEFAULT_BATCH_SIZE = 100

data/lib/sidekiq_publisher/{railtie.rb → engine.rb}

@@ -1,10 +1,8 @@
 # frozen_string_literal: true
 
 module SidekiqPublisher
-  class Railtie < Rails::Railtie
-    rake_tasks do
-      load "sidekiq_publisher/tasks.rake"
-    end
+  class Engine < Rails::Engine
+    isolate_namespace SidekiqPublisher
 
     initializer "sidekiq_publisher.configure" do
       SidekiqPublisher.logger = Rails.logger

data/lib/sidekiq_publisher/metrics_reporter.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require "active_support/subscriber"
+require "active_support/core_ext/object/try"
 
 module SidekiqPublisher
   module MetricsReporter

data/lib/sidekiq_publisher/publisher.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 require "sidekiq_publisher/client"
-require "active_support/core_ext/object/try"
+require "active_support/core_ext/object/blank"
 
 module SidekiqPublisher
   class Publisher

data/lib/sidekiq_publisher/report_unpublished_count.rb

@@ -1,10 +1,22 @@
 # frozen_string_literal: true
 
+require "active_record"
+require "pg"
+
 module SidekiqPublisher
   module ReportUnpublishedCount
     def self.call(instrumenter: Instrumenter.new)
       instrumenter.instrument("unpublished.reporter",
                               unpublished_count: SidekiqPublisher::Job.unpublished.count)
+    rescue ActiveRecord::StatementInvalid, PG::UnableToSend, PG::ConnectionBad => e
+      ActiveRecord::Base.clear_active_connections! if db_connection_error?(e)
+      raise
+    end
+
+    def self.db_connection_error?(error)
+      cause = error.is_a?(ActiveRecord::StatementInvalid) ? error.cause : error
+      cause.is_a?(PG::UnableToSend) || cause.is_a?(PG::ConnectionBad)
     end
+    private_class_method :db_connection_error?
   end
 end

data/lib/sidekiq_publisher/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module SidekiqPublisher
-  VERSION = "1.7.0.rc1"
+  VERSION = "2.0.0.rc0"
 end

data/lib/{sidekiq_publisher/tasks.rake → tasks/sidekiq_publisher.rake}

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 namespace :sidekiq_publisher do
+  desc "Start the Sidekiq Publisher runner"
   task publish: [:environment] do
     Signal.trap("INT") { exit(0) }
     Signal.trap("TERM") { exit(0) }

data/sidekiq_publisher.gemspec

@@ -41,6 +41,8 @@ Gem::Specification.new do |spec|
   spec.executables   = []
   spec.require_paths = ["lib"]
 
+  spec.required_ruby_version = ">= 2.6"
+
   spec.add_development_dependency "activejob"
   spec.add_development_dependency "appraisal"
   spec.add_development_dependency "bundler", "~> 1.12"
@@ -58,6 +60,6 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "simplecov", "< 0.18"
 
   spec.add_runtime_dependency "activerecord-postgres_pub_sub", ">= 0.4.0"
-  spec.add_runtime_dependency "activesupport", ">= 5.1", "< 6.1"
+  spec.add_runtime_dependency "activesupport", ">= 5.1", "< 6.2"
   spec.add_runtime_dependency "sidekiq", ">= 5.0.4", "< 7.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sidekiq_publisher
 version: !ruby/object:Gem::Version
-  version: 1.7.0.rc1
+  version: 2.0.0.rc0
 platform: ruby
 authors:
 - ezCater, Inc
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-08-07 00:00:00.000000000 Z
+date: 2021-04-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activejob
@@ -243,7 +243,7 @@ dependencies:
         version: '5.1'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '6.1'
+        version: '6.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -253,7 +253,7 @@ dependencies:
         version: '5.1'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '6.1'
+        version: '6.2'
 - !ruby/object:Gem::Dependency
   name: sidekiq
   requirement: !ruby/object:Gem::Requirement
@@ -289,8 +289,8 @@ files:
 - Gemfile
 - LICENSE.txt
 - README.md
+- app/models/sidekiq_publisher/job.rb
 - docker-compose.yml
-- gemfiles/rails_5.1.gemfile
 - gemfiles/rails_5.2.gemfile
 - gemfiles/rails_5.2_sidekiq_5.0.gemfile
 - gemfiles/rails_5.2_sidekiq_5.1.gemfile
@@ -300,24 +300,26 @@ files:
 - gemfiles/rails_6.0_sidekiq_5.2.gemfile
 - gemfiles/rails_6.0_sidekiq_6.0.gemfile
 - gemfiles/rails_6.0_sidekiq_6.1.gemfile
+- gemfiles/rails_6.1_sidekiq_5.2.gemfile
+- gemfiles/rails_6.1_sidekiq_6.0.gemfile
+- gemfiles/rails_6.1_sidekiq_6.1.gemfile
 - lib/active_job/queue_adapters/sidekiq_publisher_adapter.rb
 - lib/generators/sidekiq_publisher/install_generator.rb
 - lib/generators/sidekiq_publisher/templates/create_sidekiq_publisher_jobs.rb
 - lib/sidekiq_publisher.rb
 - lib/sidekiq_publisher/client.rb
 - lib/sidekiq_publisher/datadog_apm.rb
+- lib/sidekiq_publisher/engine.rb
 - lib/sidekiq_publisher/exception_reporter.rb
 - lib/sidekiq_publisher/instrumenter.rb
-- lib/sidekiq_publisher/job.rb
 - lib/sidekiq_publisher/metrics_reporter.rb
 - lib/sidekiq_publisher/publisher.rb
-- lib/sidekiq_publisher/railtie.rb
 - lib/sidekiq_publisher/report_unpublished_count.rb
 - lib/sidekiq_publisher/runner.rb
-- lib/sidekiq_publisher/tasks.rake
 - lib/sidekiq_publisher/testing.rb
 - lib/sidekiq_publisher/version.rb
 - lib/sidekiq_publisher/worker.rb
+- lib/tasks/sidekiq_publisher.rake
 - log/.gitkeep
 - sidekiq_publisher.gemspec
 homepage: https://github.com/ezcater/sidekiq_publisher
@@ -333,14 +335,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.6'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">"
     - !ruby/object:Gem::Version
       version: 1.3.1
 requirements: []
-rubygems_version: 3.1.4
+rubyforge_project:
+rubygems_version: 2.7.6.2
 signing_key:
 specification_version: 4
 summary: Publisher for enqueuing jobs to Sidekiq