sequel-activerecord_connection 1.4.3 → 1.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1459c431ad6f922b5c4400d85ae916e5032cab36a959041362f45817b604919d
-  data.tar.gz: 667fe2896a48d5d93d16e5c5b00e0d064c43d2687ef077bc9421e3d7869a41b6
+  metadata.gz: ecd95c01a950e81d89914199827c09d3ef0b95b106cfe0601afa9dd40f395f62
+  data.tar.gz: 9f5c69f4291dfd46a1d07476a1fca226aafcdd6e6ef5c0ce697554a3c2d385e3
 SHA512:
-  metadata.gz: c9f5cb40548c059e3319be0088b8a158dd66c18543fd31f1954db8bc848c4ef541ec1f779acc637211d730dfe4a080a805ee477f128474c9894cc6883e10f46d
-  data.tar.gz: 6068798de2efef62ae54cde36cb2d7abbad316aadc3eba6d5dceae1708e1aec36ec43b51cbeaf0f59609fd2be09e5d02b0f07afde5f646ebf39fb992e054b046
+  metadata.gz: 85877ab167fbca19e25b366c05f934488f610f23b790cdd360c80e67cf0cecce35104846807f77374671d6e8214008c5aaab88f5ce10d47106060e049dc94f59
+  data.tar.gz: d25ed24c5444c0c6cdfeb91857b42db90f8d3115f1ca678b4e7ec876845255d65737e8e5d2837c633058356ad7ac32ab5da71085c68c5021134765a19a10b7ee

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## HEAD
+
+* Avoid permanent checkout of Active Record 7.2+ connections (@janko)
+
 ## 1.4.3 (2024-09-26)
 
 * Fix compatibility with adapters that don't support savepoints (@janko)

data/lib/sequel/extensions/activerecord_connection/postgres.rb

@@ -25,10 +25,12 @@ module Sequel
           fail Error, "#{key.inspect} transaction option is currently not supported" if opts.key?(key)
         end
 
-        super
-      rescue => e
-        activerecord_connection.clear_cache! if e.class.name == "ActiveRecord::PreparedStatementCacheExpired" && !in_transaction?
-        raise
+        super do |conn|
+          yield conn
+        rescue => e
+          activerecord_connection.clear_cache! if e.class.name == "ActiveRecord::PreparedStatementCacheExpired"
+          raise
+        end
       end
 
       private

data/lib/sequel/extensions/activerecord_connection.rb

@@ -39,7 +39,7 @@ module Sequel
 
     # Avoid calling Sequel's connection pool, instead use Active Record's.
     def synchronize(*)
-      activerecord_lock do
+      activerecord_synchronize do
         conn = activerecord_connection.raw_connection
 
         if activerecord_connection_class && !conn.is_a?(activerecord_connection_class)
@@ -47,9 +47,9 @@ module Sequel
         end
 
         yield conn
+      ensure
+        clear_activerecord_query_cache
       end
-    ensure
-      clear_activerecord_query_cache
     end
 
     # Log executed queries into Active Record logger as well.
@@ -140,7 +140,7 @@ module Sequel
       end
     end
 
-    if ActiveRecord.version >= Gem::Version.new("7.2.0")
+    if ActiveRecord.version >= Gem::Version.new("7.2")
       def activerecord_transaction_callback(type, &block)
         activerecord_connection.current_transaction.public_send(type, &block)
       end
@@ -158,6 +158,14 @@ module Sequel
       super
     end
 
+    def activerecord_synchronize
+      with_activerecord_connection do
+        activerecord_lock do
+          yield
+        end
+      end
+    end
+
     if ActiveRecord.version >= Gem::Version.new("7.0")
       def clear_activerecord_query_cache
         activerecord_model.clear_query_caches_for_current_thread
@@ -168,10 +176,22 @@ module Sequel
       end
     end
 
+    if ActiveRecord.version >= Gem::Version.new("7.2")
+      def with_activerecord_connection
+        activerecord_model.with_connection(prevent_permanent_checkout: true) do
+          yield activerecord_connection
+        end
+      end
+    else
+      def with_activerecord_connection
+        yield activerecord_connection
+      end
+    end
+
     # Active Record doesn't guarantee that a single connection can only be used
     # by one thread at a time, so we need to use locking, which is what Active
     # Record does internally as well.
-    if ActiveRecord.version >= Gem::Version.new("5.1.0")
+    if ActiveRecord.version >= Gem::Version.new("5.1")
       def activerecord_lock
         activerecord_connection.lock.synchronize do
           ActiveSupport::Dependencies.interlock.permit_concurrent_loads do

data/sequel-activerecord_connection.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = "sequel-activerecord_connection"
-  spec.version       = "1.4.3"
+  spec.version       = "1.5.0"
   spec.authors       = ["Janko Marohnić"]
   spec.email         = ["janko.marohnic@gmail.com"]
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sequel-activerecord_connection
 version: !ruby/object:Gem::Version
-  version: 1.4.3
+  version: 1.5.0
 platform: ruby
 authors:
 - Janko Marohnić
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-09-26 00:00:00.000000000 Z
+date: 2024-10-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sequel