pgbus 0.11.3 → 0.12.0

data/lib/pgbus/process/notify_listener.rb

@@ -146,7 +146,9 @@ module Pgbus
           wait_once
         end
       rescue StandardError => e
-        @logger.error { "[Pgbus::NotifyListener] fatal: #{e.class}: #{e.message}" } if running?
+        # Report, don't just log: a listener that dies at boot silently
+        # degrades every worker to polling (issue #352).
+        ErrorReporter.report(e, { action: "notify_listener_fatal" }) if running?
       ensure
         # Clear @running so #start can spawn a fresh thread after a fatal exit
         # (e.g. build_connection raising at boot). Without this, the dead
@@ -242,7 +244,7 @@ module Pgbus
             # partially-built conn is orphaned and the next retry just allocates
             # another one — leaking PG connections on repeated failures.
             close_quietly(new_conn)
-            @logger.error { "[Pgbus::NotifyListener] reconnect failed: #{e.class}: #{e.message}" }
+            ErrorReporter.report(e, { action: "notify_listener_reconnect" })
             sleep RECONNECT_BACKOFF_SECONDS
             next
           end
@@ -255,11 +257,11 @@ module Pgbus
         end
       end
 
+      # String/Hash go through DedicatedConnection so a :session-mode
+      # `:variables` key never reaches PG.connect (issue #352).
       def build_connection
-        require "pg" unless defined?(::PG::Connection)
         case @connection_options
-        when String then ::PG.connect(@connection_options)
-        when Hash   then ::PG.connect(**@connection_options)
+        when String, Hash then Pgbus::DedicatedConnection.connect(@connection_options)
         else
           raise Pgbus::ConfigurationError,
                 "NotifyListener cannot build a PG connection from #{@connection_options.class}. " \

data/lib/pgbus/version.rb

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

data/lib/pgbus/web/stream_app.rb

@@ -74,7 +74,11 @@ module Pgbus
           unsupported_server
         end
       rescue StandardError => e
-        logger.error { "[Pgbus::StreamApp] #{e.class}: #{e.message}" }
+        # Report, don't just log: a broken streamer config (e.g. bad
+        # streams_connection_options) 500s EVERY stream request on this
+        # worker — logger-only made that invisible to APM (issue #352).
+        ErrorReporter.report(e, { action: "stream_connect" }, config: config)
+        logger.debug { e.backtrace&.join("\n") }
         server_error
       end
 

data/lib/pgbus/web/streamer/instance.rb

@@ -230,16 +230,16 @@ module Pgbus
           probe(validate_primary(build_raw_pg_connection))
         end
 
-        # Raw PG.connect with no validation or probe. This is what the Listener's
+        # Raw connect with no validation or probe. This is what the Listener's
         # connection_factory calls on every reconnect attempt: the reconnect loop
         # runs its own PrimaryValidator (retrying on a replica) and skips the
         # probe to stay cheap, mirroring Pgbus::Process::NotifyListener.
+        # String/Hash go through DedicatedConnection so a :session-mode
+        # `:variables` key never reaches PG.connect (issue #352).
         def build_raw_pg_connection
-          require "pg" unless defined?(::PG::Connection)
           opts = @config.streams_connection_options
           case opts
-          when String then ::PG.connect(opts)
-          when Hash   then ::PG.connect(**opts)
+          when String, Hash then Pgbus::DedicatedConnection.connect(opts)
           when Proc
             # The Proc branch in connection_options typically returns
             # ActiveRecord::Base.connection.raw_connection — a pooled

data/lib/pgbus/web/streamer/listener.rb

@@ -303,7 +303,7 @@ module Pgbus
               # don't leak PG connections, then back off and retry rather than
               # letting the exception kill the listener thread silently.
               close_quietly(new_conn)
-              @logger.error { "[Pgbus::Streamer::Listener] reconnect failed: #{e.class}: #{e.message}" }
+              Pgbus::ErrorReporter.report(e, { action: "streamer_reconnect" })
               sleep RECONNECT_BACKOFF_SECONDS
               next
             end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pgbus
 version: !ruby/object:Gem::Version
-  version: 0.11.3
+  version: 0.12.0
 platform: ruby
 authors:
 - Mikael Henriksson
@@ -63,14 +63,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.7.0
+        version: 0.7.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.7.0
+        version: 0.7.1
 - !ruby/object:Gem::Dependency
   name: railties
   requirement: !ruby/object:Gem::Requirement
@@ -266,6 +266,7 @@ files:
 - lib/pgbus/concurrency/semaphore.rb
 - lib/pgbus/configuration.rb
 - lib/pgbus/configuration/capsule_dsl.rb
+- lib/pgbus/dedicated_connection.rb
 - lib/pgbus/dedup_cache.rb
 - lib/pgbus/doctor.rb
 - lib/pgbus/engine.rb
@@ -320,6 +321,7 @@ files:
 - lib/pgbus/pgmq_schema.rb
 - lib/pgbus/pgmq_schema/pgmq_v1.11.0.sql
 - lib/pgbus/pgmq_schema/pgmq_v1.11.1.sql
+- lib/pgbus/pgmq_schema/pgmq_v1.12.0.sql
 - lib/pgbus/process/consumer.rb
 - lib/pgbus/process/consumer_priority.rb
 - lib/pgbus/process/dispatcher.rb