pgbus 0.7.6 → 0.7.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a28025e2b7af463cb6ff57d12e516d461b9b968ed7bb4eb4568be18d77ff2678
-  data.tar.gz: fd39f5774df158b0cf06e85a6ef52f0114d649008f56b4736a782ce895baae0b
+  metadata.gz: 48e02fc16de64a9ccec20b8b0d917e154e17dc2082c1e3933b80da1bdc2f0c2a
+  data.tar.gz: 811db2dfcb4c4271296d9125754af3865b117ebc8b45d3194bc4314cc43ffaba
 SHA512:
-  metadata.gz: 19ced25de78f93ccbf307e2bdd72a5dbc46050657c6e394bc9e6a16d8665fa19bc2a29cebf2fcbda8c31eb07e07d94545322eec80c5d61915ddd6bfc924de105
-  data.tar.gz: 9d8e2a9255ce011edf6bc2647fbd4d604db65b8fb2470c17735149599072f2824e510db4e97c4d241cdd4d6a9fef2ff7c23638738cd40de581ee56a07b783f6b
+  metadata.gz: 2b41cd760d469af41c4e0ef0d9bfaffd3cb17ff81b38dbeb250605d78c737708f54ee43519fe25afeda83a5e15177a14e89e48d27471ae39ad5c79420d791e21
+  data.tar.gz: 50f209ed7b74e18c1bf927ca73fe80286c9a4333642af4f32e579bd6f9e40158007c07a39dfa6a3fc2082fd9ad1663f3e9b109ebd7021485f931a615ad9baabb

data/CHANGELOG.md

@@ -15,6 +15,7 @@
 
 ### Fixed
 
+- **EventBus handler invocations now run inside `Rails.application.executor` (or `.reloader` in dev).** `Pgbus::EventBus::Handler#process` previously ran outside any Rails executor wrapper, so `claim_idempotency?`'s `ProcessedEvent.insert` (and any `handle` body that touches AR) leased a connection that was never returned to the pool. With a small AR pool (Rails dev default with `RAILS_MAX_THREADS=3` is 6 connections), a handful of consumed events leaked the entire pool. The next request that triggers `clear_reloadable_connections!` (any reloadable change in dev) hung in `with_exclusively_acquired_all_connections`, surfacing as a confusing `Rack::Timeout` with a trace ending in `MonitorMixin#wait_for_cond`. Mirrors the executor-wrap pattern already used by `Pgbus::ActiveJob::Executor#execute_job`. No-op when Rails isn't loaded.
 - **Defensive retry on stale pooled pgmq connections in the enqueue path.** `Pgbus::Client#send_message`, `#send_batch`, and `#publish_to_topic` now retry once when `@pgmq.produce*` raises `PGMQ::Errors::ConnectionError` with a message indicating the pooled `PG::Connection` was killed beneath pgmq-ruby — typically by PgBouncer hitting `server_idle_timeout` / `client_idle_timeout`, an admin disconnect, or a TCP RST. Observed in production as `PQsocket() can't get socket descriptor` on the first produce following an idle window. pgmq-ruby's `auto_reconnect` recovers on the *next* pool checkout, so a single retry is sufficient; non-stale errors (pool timeout, misconfiguration, unreachable database) still propagate unchanged. Upstream pgmq-ruby fix for the underlying misclassification is in-flight at mensfeld/pgmq-ruby#94.
 
 ## [0.5.1] - 2026-04-08

data/lib/pgbus/active_job/executor.rb

@@ -107,12 +107,22 @@ module Pgbus
 
       def execute_job(job)
         if defined?(Rails) && Rails.respond_to?(:application) && Rails.application
-          Rails.application.executor.wrap { job.perform_now }
+          wrapper = reloading? ? Rails.application.reloader : Rails.application.executor
+          wrapper.wrap { job.perform_now }
         else
           job.perform_now
         end
       end
 
+      def reloading?
+        app_config = Rails.application.config
+        if app_config.respond_to?(:enable_reloading)
+          app_config.enable_reloading
+        else
+          !app_config.cache_classes
+        end
+      end
+
       def monotonic_now
         ::Process.clock_gettime(::Process::CLOCK_MONOTONIC)
       end

data/lib/pgbus/event_bus/handler.rb

@@ -18,6 +18,16 @@ module Pgbus
       end
 
       def process(message)
+        with_rails_executor { process!(message) }
+      end
+
+      def handle(event)
+        raise NotImplementedError, "#{self.class.name} must implement #handle(event)"
+      end
+
+      private
+
+      def process!(message)
         raw = JSON.parse(message.message)
         event = build_event(raw)
 
@@ -28,11 +38,30 @@ module Pgbus
         :handled
       end
 
-      def handle(event)
-        raise NotImplementedError, "#{self.class.name} must implement #handle(event)"
+      # Mirrors Pgbus::ActiveJob::Executor#execute_job: wrap the handler
+      # invocation in Rails.application.executor (or the reloader in dev)
+      # so AR connections leased by `claim_idempotency?` and `handle` are
+      # released back to the pool when this method returns. Without the
+      # wrap, every consumed event leaks one AR connection on the consumer
+      # thread — in dev that wedges `clear_reloadable_connections!`,
+      # producing a confusing Rack::Timeout in `MonitorMixin#wait_for_cond`.
+      #
+      # No-op when Rails isn't loaded (test harnesses, gem-only consumers).
+      def with_rails_executor(&)
+        return yield unless defined?(Rails) && Rails.respond_to?(:application) && Rails.application
+
+        wrapper = reloading? ? Rails.application.reloader : Rails.application.executor
+        wrapper.wrap(&)
       end
 
-      private
+      def reloading?
+        app_config = Rails.application.config
+        if app_config.respond_to?(:enable_reloading)
+          app_config.enable_reloading
+        else
+          !app_config.cache_classes
+        end
+      end
 
       def build_event(raw)
         payload = raw["payload"]

data/lib/pgbus/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Pgbus
-  VERSION = "0.7.6"
+  VERSION = "0.7.8"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pgbus
 version: !ruby/object:Gem::Version
-  version: 0.7.6
+  version: 0.7.8
 platform: ruby
 authors:
 - Mikael Henriksson