active_record_host_pool 3.1.1 → 4.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e9937e04d8be8861b4e568b680048b61bacb86e8a2c8ce0a459943c44640f3af
-  data.tar.gz: cb20c2fefd723228aa9a97b9578fe121b4b1c8230e6309be8791e9ddf313bfc2
+  metadata.gz: 8149f10e7b7b05c8ce36cb00ee2c0a77c6df40df5e594eb66cdcfbd42c265ddf
+  data.tar.gz: d9464ea5f9f1d310259c54b22d0d7df40c8b04d0a0568d378893174861675f7b
 SHA512:
-  metadata.gz: 576feb76ce2e40a7a514c8f520fd9b15ea4cae69f7126b0124a1d418b16777ff8b70cae9381c11c99e95484e045f09879190282b9c015cb98438907770a60518
-  data.tar.gz: dc3f61f9dc3598a6c563d291885fd3723ae9266640037095238f5cdced37a0e128deaca5e9415b850a47fa3b9a35339fedcaf5d8657bf015324b350337b9b773
+  metadata.gz: e5454f3f23ffd85a74e0fea41588b4b4093578893dfbf4d4c7e25a18b05ea45749249acbf9cb0755d1921c37c2e3016c8ec006171c21d4e8f79cee25488c0ba2
+  data.tar.gz: c8e2aa410f402b5272d37fd2c6e0405b4771469325c660e865f22a8adffcb0d6f9a1c574bab9952b81c81ef0186df9884042e35564a91cdc23b89ba5bf85f120

data/Changelog.md

@@ -6,6 +6,20 @@ and as of v1.0.0 this project adheres to [Semantic Versioning](https://semver.or
 
 ## [Unreleased]
 
+## [4.0.0]
+
+### Changed
+- Moved `select_db` inside of the `with_raw_connection` block of the `#raw_execute` method. This should allow for using Rails' built-in reconnect & retry logic with the Trilogy adapter or Rails 7.1+.
+- In Rails 7.1+, when a new ConnectionProxy is instantiated the database switch is lazily triggered by the subsequent database query instead of immediately.
+
+### Removed
+- Calling `#clean!` and `#verified!` on connections because it is no longer necessary.
+
+## [3.2.0]
+
+### Added
+- Calls `#verified!` on the connection after `#clean!`.
+
 ## [3.1.1]
 
 ### Fixed

data/lib/active_record_host_pool/connection_adapter_mixin.rb

@@ -28,20 +28,23 @@ module ActiveRecordHostPool
       @config[:database] = _host_pool_desired_database
     end
 
-    if ActiveRecord.version >= Gem::Version.new("7.1") || ActiveRecordHostPool.loaded_db_adapter == :trilogy
-      # Patch `raw_execute` instead of `execute` since this commit:
-      # https://github.com/rails/rails/commit/f69bbcbc0752ca5d5af327d55922614a26f5c7e9
-      def raw_execute(...)
-        if _host_pool_desired_database && !_no_switch
-          _switch_connection
+    if ActiveRecord.version >= Gem::Version.new("7.1")
+      def with_raw_connection(...)
+        super do |real_connection|
+          _switch_connection(real_connection) if _host_pool_desired_database && !_no_switch
+          yield real_connection
+        end
+      end
+    elsif ActiveRecordHostPool.loaded_db_adapter == :trilogy
+      def with_trilogy_connection(...)
+        super do |real_connection|
+          _switch_connection(real_connection) if _host_pool_desired_database && !_no_switch
+          yield real_connection
         end
-        super
       end
     else
       def execute(...)
-        if _host_pool_desired_database && !_no_switch
-          _switch_connection
-        end
+        _switch_connection(raw_connection) if _host_pool_desired_database && !_no_switch
         super
       end
     end
@@ -70,7 +73,7 @@ module ActiveRecordHostPool
 
     attr_accessor :_no_switch
 
-    def _switch_connection
+    def _switch_connection(real_connection)
       if _host_pool_desired_database &&
           (
            _desired_database_changed? ||
@@ -78,8 +81,7 @@ module ActiveRecordHostPool
          )
         log("select_db #{_host_pool_desired_database}", "SQL") do
           clear_cache!
-          raw_connection.select_db(_host_pool_desired_database)
-          clean! if respond_to?(:clean!)
+          real_connection.select_db(_host_pool_desired_database)
         end
         @_cached_current_database = _host_pool_desired_database
         @_cached_connection_object_id = _real_connection_object_id

data/lib/active_record_host_pool/pool_proxy.rb

@@ -219,12 +219,7 @@ module ActiveRecordHostPool
         @connection_proxy_cache ||= {}
         key = [connection, database]
 
-        @connection_proxy_cache[key] ||= begin
-          cx = ActiveRecordHostPool::ConnectionProxy.new(connection, database)
-          cx.raw_execute("SELECT 1", "ARHP SWITCH DB")
-
-          cx
-        end
+        @connection_proxy_cache[key] ||= ActiveRecordHostPool::ConnectionProxy.new(connection, database)
       end
     end
     # standard:enable Lint/DuplicateMethods

data/lib/active_record_host_pool/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ActiveRecordHostPool
-  VERSION = "3.1.1"
+  VERSION = "4.0.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: active_record_host_pool
 version: !ruby/object:Gem::Version
-  version: 3.1.1
+  version: 4.0.0
 platform: ruby
 authors:
 - Benjamin Quorning
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-09-02 00:00:00.000000000 Z
+date: 2024-09-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -66,7 +66,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.11
+rubygems_version: 3.5.16
 signing_key: 
 specification_version: 4
 summary: Allow ActiveRecord to share a connection to multiple databases on the same