sequel-activerecord_connection 1.2.6 → 1.2.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: aa32893e39b55c6e2b8186c216645a4cccfa7e865795e390656ebac83d5ecd1d
-  data.tar.gz: c8d6ba5b2958de6e8e3d687c75f4a51a60c863a1728de525ef222650cd34b490
+  metadata.gz: 9cd2f9d2df2e116227a94f6a005a5f7470178e4ecfc22218fe0a3dc456a2ecd2
+  data.tar.gz: 64e96f773827d185768d6190468a350162c3e7e4923a5f5dc565d50ec8064f79
 SHA512:
-  metadata.gz: fba2ec585154bfd9477b2ace86b7c6b6b36cc5b4ba11f21a62bde593b1c8069858fb33d4984dced898b02a831f75fd2219c43adb26d67b93d17ce0313965c30a
-  data.tar.gz: 685f0dd698849465d1902ee7cd4432d2de6ea25f7791707a6defae8f32ff0e66713a4f391573924e79dfe3f91723fa564083b1dbefbf2c2986047205fe024aec
+  metadata.gz: 58862596df481839c1668ce4f3062f56255dc5115a2aa8bfb08499c10b76eac0859b51485f639c5fa16459cff170c2531b7c86c0e38efe9d5dfe6bc94a6a2891
+  data.tar.gz: 1aa33eb5cd5eb85be84252bd3ddd02ec537e7bf89481882604fc691d72d055d4bd85e6ad97562c44b39c90beabb661ba04ba1a311775b707aa71984f23dc8e25

data/CHANGELOG.md

@@ -1,6 +1,18 @@
+## 1.2.9 (2022-03-15)
+
+* Remove `sequel_pg` and `pg` runtime dependencies introduced in the previous version (@janko)
+
+## 1.2.8 (2022-02-28)
+
+* Support the pg_streaming database extension from the sequel_pg gem (@janko)
+
+## 1.2.7 (2022-01-20)
+
+* Require Sequel 3.38+ (@janko)
+
 ## 1.2.6 (2021-12-26)
 
-* Speed up `#activerecord_lock` by avoiding checking Active Record version at runtime (@janko)
+* Speed up connection access by avoiding checking Active Record version at runtime (@janko)
 
 ## 1.2.5 (2021-12-19)
 

data/LICENSE.txt

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright (c) 2020 Janko Marohnić
+Copyright (c) 2020-2022 Janko Marohnić
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

data/README.md

@@ -1,18 +1,42 @@
 # sequel-activerecord_connection
 
-This is an extension for [Sequel] that allows it to reuse an existing
-ActiveRecord connection for database interaction.
+This is a database extension for [Sequel] that makes it to reuse an existing
+Active Record connection for database interaction.
 
-This can be useful if you're using a library that uses Sequel for database
-interaction (e.g. [Rodauth] or [rom-sql]), but you want to avoid creating a
-separate database connection. Or if you're transitioning from ActiveRecord to
-Sequel, and want the database connection to be shared.
+This can be useful if you want to use a library that uses Sequel (e.g.
+[Rodauth] or [rom-sql]), or you're transitioning from Active Record to Sequel,
+or if you just want to use Sequel for more complex queries, and you want to
+avoid creating new database connections.
 
 It works on ActiveRecord 4.2+ and fully supports PostgresSQL, MySQL and SQLite
-adapters, both the native ones and JDBC (JRuby). There is attempted suppport
-for [Oracle enhanced] and [SQL Server] Active Record adapters (`oracle` and
-`tinytds` in Sequel). Other adapters might work too, but their integration
-hasn't been tested.
+adapters, both native and JDBC (JRuby). There is attempted suppport for [Oracle
+enhanced] and [SQL Server] Active Record adapters (`oracle` and `tinytds` in
+Sequel). Other adapters might work too, but their integration hasn't been
+tested.
+
+## Why reuse the database connection?
+
+At first it might appear that, as long as you're fine with the performance
+impact of your database server having to maintain additional open connections,
+it would be fine if Sequel had its own database connection. However, there are
+additional caveats when you try to combine it with Active Record.
+
+If Sequel and Active Record each have their own connections, then it's not
+possible to combine their transactions. If we executed a Sequel query inside of
+an Active Record transaction, that query won't actually be executed inside a
+database transaction. This is because transactions are tied to the database
+connection; if one connection opens a transaction, this doesn't affect queries
+executed on a different connection, even if both connections are used in the
+same ruby process. With this library, transactions and queries can be
+seamlessly combined between Active Record and Sequel.
+
+In Rails context, there are additional considerations for a Sequel connection
+to play nicely. Connecting and disconnecting would have to go in lockstep with
+Active Record, to make commands such as `rails db:create` and `rails db:drop`
+work. You'd also need to find a way for system tests and the app running in the
+background to share the same database connection, which is something Sequel
+wasn't designed for. Reusing Active Record's connection means (dis)connecting
+and sharing between threads is all handled automatically.
 
 ## Installation
 
@@ -140,9 +164,8 @@ DB.transaction(auto_savepoint: true) do
 end
 #>> BEGIN
 #>> SAVEPOINT active_record_1
-#>> RELEASE SAVEPOINT active_record_1
+#>> ROLLBACK TO SAVEPOINT active_record_1
 #>> COMMIT
-#>> after commit
 ```
 
 In case of (a) adding a transaction hook while Active Record holds the
@@ -168,13 +191,13 @@ end
 # after a savepoint is released, if the enclosing transaction is not joinable.
 ActiveRecord::Base.transaction(joinable: false) do
   DB.transaction do
-    DB.after_commit { puts "after commit" }
+    DB.after_commit { puts "after savepoint release" }
   end
 end
 #>> BEGIN
 #>> SAVEPOINT active_record_1
 #>> RELEASE SAVEPOINT active_record_1
-#>> after commit
+#>> after savepoint release
 #>> COMMIT
 ```
 

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

@@ -10,6 +10,11 @@ module Sequel
 
           Utils.add_prepared_statements_cache(conn)
 
+          # compatibility for pg_streaming database extension from sequel_pg gem
+          if defined?(Sequel::Postgres::Streaming) && is_a?(Sequel::Postgres::Streaming)
+            conn.extend(Sequel::Postgres::Streaming::AdapterMethods)
+          end
+
           yield conn
         end
       end
@@ -70,11 +75,11 @@ module Sequel
         # yield the results, otherwise, return the number of changed rows.
         def execute(sql, args = nil)
           args   = args.map { |v| @db.bound_variable_arg(v, self) } if args
-          result = check_disconnect_errors { execute_query(sql, args) }
+          q = check_disconnect_errors { execute_query(sql, args) }
 
-          block_given? ? yield(result) : result.cmd_tuples
+          block_given? ? yield(q) : q.cmd_tuples
         ensure
-          result.clear if result
+          q.clear if q && q.respond_to?(:clear)
         end
 
         private

data/sequel-activerecord_connection.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = "sequel-activerecord_connection"
-  spec.version       = "1.2.6"
+  spec.version       = "1.2.9"
   spec.authors       = ["Janko Marohnić"]
   spec.email         = ["janko.marohnic@gmail.com"]
 
@@ -11,11 +11,11 @@ Gem::Specification.new do |spec|
 
   spec.required_ruby_version = ">= 2.4"
 
-  spec.add_dependency "sequel", "~> 5.16"
+  spec.add_dependency "sequel", "~> 5.38"
   spec.add_dependency "activerecord", ">= 4.2", "< 8"
   spec.add_dependency "after_commit_everywhere", "~> 1.1"
 
-  spec.add_development_dependency "sequel", "~> 5.38"
+  spec.add_development_dependency "sequel_pg" unless RUBY_ENGINE == "jruby"
   spec.add_development_dependency "minitest"
   spec.add_development_dependency "warning"
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sequel-activerecord_connection
 version: !ruby/object:Gem::Version
-  version: 1.2.6
+  version: 1.2.9
 platform: ruby
 authors:
 - Janko Marohnić
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-12-26 00:00:00.000000000 Z
+date: 2022-03-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sequel
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.16'
+        version: '5.38'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.16'
+        version: '5.38'
 - !ruby/object:Gem::Dependency
   name: activerecord
   requirement: !ruby/object:Gem::Requirement
@@ -59,19 +59,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1.1'
 - !ruby/object:Gem::Dependency
-  name: sequel
+  name: sequel_pg
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '5.38'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '5.38'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
@@ -138,7 +138,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.15
+rubygems_version: 3.3.3
 signing_key:
 specification_version: 4
 summary: Allows Sequel to use ActiveRecord connection for database interaction.