sequel-activerecord_connection 0.3.0 → 0.4.1
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/CHANGELOG.md +18 -0
- data/README.md +19 -23
- data/lib/sequel/extensions/activerecord_connection.rb +43 -24
- data/sequel-activerecord_connection.gemspec +2 -2
- metadata +6 -12
checksums.yaml
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
---
|
2
2
|
SHA256:
|
3
|
-
metadata.gz:
|
4
|
-
data.tar.gz:
|
3
|
+
metadata.gz: bb35cdbc0ade81ae15042834733f9ec4683b6889eab2c35412710d631a02de53
|
4
|
+
data.tar.gz: 75f97d5bee8522c12a7a3c65e05e1511f40ca4eccab8bc4b8cb2e1c3ece1a44a
|
5
5
|
SHA512:
|
6
|
-
metadata.gz:
|
7
|
-
data.tar.gz:
|
6
|
+
metadata.gz: 4fbf8edd6c9a61a8761431d33ba89c63a2aa444417d5ebec7b42ec9d3ef8aa5c509b28d35d708af223289c8e085b62204c3af690dc18c13e0df69f61a837b41d
|
7
|
+
data.tar.gz: 5f4408be86e1d09a856215ca6e1fd4ef594955549e52987107ff33fc8efb24c68f4fdaf03c0132bb22fe4e2023b31d5526dc77f225d476718528d101c2f6bb8c
|
data/CHANGELOG.md
CHANGED
@@ -1,3 +1,21 @@
|
|
1
|
+
## 0.4.1 (2020-09-28)
|
2
|
+
|
3
|
+
* Require Sequel version 5.16.0 or above (@janko)
|
4
|
+
|
5
|
+
## 0.4.0 (2020-09-28)
|
6
|
+
|
7
|
+
* Return correct result of `Database#in_transaction?` after ActiveRecord transaction exited (@janko)
|
8
|
+
|
9
|
+
* Make ActiveRecord create a savepoint inside a Sequel transaction with `auto_savepoint: true` (@janko)
|
10
|
+
|
11
|
+
* Make Sequel create a savepoint inside ActiveRecord transaction with `joinable: false` (@janko)
|
12
|
+
|
13
|
+
* Improve reliability of nested transactions when combining Sequel and ActiveRecord (@janko)
|
14
|
+
|
15
|
+
* Raise error when attempting to add an `after_commit`/`after_rollback` hook on ActiveRecord transaction (@janko)
|
16
|
+
|
17
|
+
* Fix infinite loop that could happen with transactional Rails tests (@janko)
|
18
|
+
|
1
19
|
## 0.3.0 (2020-07-24)
|
2
20
|
|
3
21
|
* Fully support Sequel transaction API (all transaction options, transaction/savepoint hooks etc.) (@janko)
|
data/README.md
CHANGED
@@ -42,8 +42,7 @@ appropriate Sequel adapter and load the `activerecord_connection` extension:
|
|
42
42
|
```rb
|
43
43
|
require "sequel"
|
44
44
|
|
45
|
-
DB = Sequel.postgres(
|
46
|
-
DB.extension :activerecord_connection
|
45
|
+
DB = Sequel.postgres(extensions: :activerecord_connection)
|
47
46
|
```
|
48
47
|
|
49
48
|
Now any Sequel operations that you make will internaly be done using the
|
@@ -75,13 +74,13 @@ ActiveRecord adapters, just make sure to initialize the corresponding Sequel
|
|
75
74
|
adapter before loading the extension.
|
76
75
|
|
77
76
|
```rb
|
78
|
-
DB = Sequel.postgres(
|
77
|
+
DB = Sequel.postgres(extensions: :activerecord_connection) # for "postgresql" adapter
|
79
78
|
# or
|
80
|
-
DB = Sequel.mysql2(
|
79
|
+
DB = Sequel.mysql2(extensions: :activerecord_connection) # for "mysql2" adapter
|
81
80
|
# or
|
82
|
-
DB = Sequel.sqlite(
|
81
|
+
DB = Sequel.sqlite(extensions: :activerecord_connection) # for "sqlite3" adapter
|
83
82
|
# or
|
84
|
-
DB = Sequel.jdbc(
|
83
|
+
DB = Sequel.jdbc(extensions: :activerecord_connection) # for JDBC adapter
|
85
84
|
```
|
86
85
|
|
87
86
|
### Transactions
|
@@ -101,39 +100,36 @@ Sequel's transaction API is fully supported:
|
|
101
100
|
|
102
101
|
```rb
|
103
102
|
DB.transaction(isolation: :serializable) do
|
104
|
-
DB.after_commit { ... } #
|
103
|
+
DB.after_commit { ... } # executed after transaction commits
|
105
104
|
DB.transaction(savepoint: true) do # creates a savepoint
|
106
|
-
#
|
105
|
+
DB.after_commit(savepoint: true) { ... } # executed if all enclosing savepoints have been released
|
107
106
|
end
|
108
107
|
end
|
109
108
|
```
|
110
109
|
|
111
|
-
One caveat to keep in mind is that Sequel's transaction hooks
|
112
|
-
|
113
|
-
|
110
|
+
One caveat to keep in mind is that using Sequel's transaction/savepoint hooks
|
111
|
+
currently don't work if ActiveRecord holds the corresponding
|
112
|
+
transaction/savepoint. This is because it's difficult to be notified when
|
113
|
+
ActiveRecord commits or rolls back the transaction/savepoint.
|
114
114
|
|
115
115
|
```rb
|
116
116
|
DB.transaction do
|
117
117
|
DB.after_commit { ... } # will get executed
|
118
118
|
end
|
119
119
|
|
120
|
-
|
121
|
-
DB.
|
120
|
+
DB.transaction do
|
121
|
+
DB.transaction(savepoint: true) do
|
122
|
+
DB.after_commit(savepoint: true) { ... } # will get executed
|
123
|
+
end
|
122
124
|
end
|
123
125
|
|
124
126
|
ActiveRecord::Base.transaction do
|
125
|
-
DB.
|
126
|
-
DB.after_commit { ... } # won't get executed
|
127
|
-
end
|
127
|
+
DB.after_commit { ... } # not allowed (will raise Sequel::ActiveRecordConnection::Error)
|
128
128
|
end
|
129
|
-
```
|
130
129
|
|
131
|
-
|
132
|
-
|
133
|
-
|
134
|
-
ActiveRecord::Base.transaction do
|
135
|
-
DB.transaction(savepoint: true) do
|
136
|
-
DB.after_commit { ... } # will get executed after savepoint is released
|
130
|
+
DB.transaction do
|
131
|
+
ActiveRecord::Base.transaction(requires_new: true) do
|
132
|
+
DB.after_commit(savepoint: true) { ... } # not allowed (will raise Sequel::ActiveRecordConnection::Error)
|
137
133
|
end
|
138
134
|
end
|
139
135
|
```
|
@@ -1,3 +1,5 @@
|
|
1
|
+
# frozen_string_literal: true
|
2
|
+
|
1
3
|
module Sequel
|
2
4
|
module ActiveRecordConnection
|
3
5
|
Error = Class.new(Sequel::Error)
|
@@ -41,38 +43,43 @@ module Sequel
|
|
41
43
|
|
42
44
|
private
|
43
45
|
|
44
|
-
#
|
45
|
-
#
|
46
|
-
#
|
47
|
-
# when to run transaction/savepoint hooks etc.
|
46
|
+
# Synchronizes transaction state with ActiveRecord. Sequel uses this
|
47
|
+
# information to know whether we're in a transaction, whether to create a
|
48
|
+
# savepoint, when to run transaction/savepoint hooks etc.
|
48
49
|
def _trans(conn)
|
49
|
-
|
50
|
-
result = @transactions[conn]
|
51
|
-
|
52
|
-
if activerecord_connection.transaction_open?
|
53
|
-
result ||= { savepoints: [] }
|
54
|
-
while result[:savepoints].length < activerecord_connection.open_transactions
|
55
|
-
result[:savepoints].unshift({ activerecord: true })
|
56
|
-
end
|
57
|
-
end
|
50
|
+
hash = super || { savepoints: [], activerecord: true }
|
58
51
|
|
59
|
-
|
60
|
-
|
52
|
+
# add any ActiveRecord transactions/savepoints that have been opened
|
53
|
+
# directly via ActiveRecord::Base.transaction
|
54
|
+
while hash[:savepoints].length < activerecord_connection.open_transactions
|
55
|
+
hash[:savepoints] << { activerecord: true }
|
56
|
+
end
|
57
|
+
|
58
|
+
# remove any ActiveRecord transactions/savepoints that have been closed
|
59
|
+
# directly via ActiveRecord::Base.transaction
|
60
|
+
while hash[:savepoints].length > activerecord_connection.open_transactions && hash[:savepoints].last[:activerecord]
|
61
|
+
hash[:savepoints].pop
|
62
|
+
end
|
63
|
+
|
64
|
+
# sync knowledge about joinability of current ActiveRecord transaction/savepoint
|
65
|
+
if activerecord_connection.transaction_open? && !activerecord_connection.current_transaction.joinable?
|
66
|
+
hash[:savepoints].last[:auto_savepoint] = true
|
67
|
+
end
|
68
|
+
|
69
|
+
if hash[:savepoints].empty? && hash[:activerecord]
|
70
|
+
Sequel.synchronize { @transactions.delete(conn) }
|
71
|
+
else
|
72
|
+
Sequel.synchronize { @transactions[conn] = hash }
|
61
73
|
end
|
62
|
-
end
|
63
74
|
|
64
|
-
# First delete any transactions/savepoints opened directly via
|
65
|
-
# ActiveRecord::Base.transaction, so that Sequel can detect when the last
|
66
|
-
# Sequel transaction has been closed and clear transaction information.
|
67
|
-
def transaction_finished?(conn)
|
68
|
-
_trans(conn)[:savepoints].shift while _trans(conn)[:savepoints].first[:activerecord]
|
69
75
|
super
|
70
76
|
end
|
71
77
|
|
72
78
|
def begin_transaction(conn, opts = {})
|
73
79
|
isolation = TRANSACTION_ISOLATION_MAP.fetch(opts[:isolation]) if opts[:isolation]
|
80
|
+
joinable = !opts[:auto_savepoint]
|
74
81
|
|
75
|
-
activerecord_connection.begin_transaction(isolation: isolation)
|
82
|
+
activerecord_connection.begin_transaction(isolation: isolation, joinable: joinable)
|
76
83
|
end
|
77
84
|
|
78
85
|
def commit_transaction(conn, opts = {})
|
@@ -84,8 +91,20 @@ module Sequel
|
|
84
91
|
activerecord_connection.transaction_manager.send(:after_failure_actions, activerecord_connection.current_transaction, $!) if activerecord_connection.transaction_manager.respond_to?(:after_failure_actions)
|
85
92
|
end
|
86
93
|
|
87
|
-
def
|
88
|
-
|
94
|
+
def add_transaction_hook(conn, type, block)
|
95
|
+
if _trans(conn)[:activerecord]
|
96
|
+
fail Error, "cannot add transaction hook when ActiveRecord holds the outer transaction"
|
97
|
+
end
|
98
|
+
|
99
|
+
super
|
100
|
+
end
|
101
|
+
|
102
|
+
def add_savepoint_hook(conn, type, block)
|
103
|
+
if _trans(conn)[:savepoints].last[:activerecord]
|
104
|
+
fail Error, "cannot add savepoint hook when ActiveRecord holds the current savepoint"
|
105
|
+
end
|
106
|
+
|
107
|
+
super
|
89
108
|
end
|
90
109
|
|
91
110
|
def activerecord_raw_connection
|
@@ -1,6 +1,6 @@
|
|
1
1
|
Gem::Specification.new do |spec|
|
2
2
|
spec.name = "sequel-activerecord_connection"
|
3
|
-
spec.version = "0.
|
3
|
+
spec.version = "0.4.1"
|
4
4
|
spec.authors = ["Janko Marohnić"]
|
5
5
|
spec.email = ["janko.marohnic@gmail.com"]
|
6
6
|
|
@@ -11,7 +11,7 @@ Gem::Specification.new do |spec|
|
|
11
11
|
|
12
12
|
spec.required_ruby_version = Gem::Requirement.new(">= 2.2.0")
|
13
13
|
|
14
|
-
spec.add_dependency "sequel", "
|
14
|
+
spec.add_dependency "sequel", "~> 5.16"
|
15
15
|
spec.add_dependency "activerecord", ">= 4.2", "< 7"
|
16
16
|
|
17
17
|
spec.add_development_dependency "minitest"
|
metadata
CHANGED
@@ -1,35 +1,29 @@
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
2
2
|
name: sequel-activerecord_connection
|
3
3
|
version: !ruby/object:Gem::Version
|
4
|
-
version: 0.
|
4
|
+
version: 0.4.1
|
5
5
|
platform: ruby
|
6
6
|
authors:
|
7
7
|
- Janko Marohnić
|
8
8
|
autorequire:
|
9
9
|
bindir: bin
|
10
10
|
cert_chain: []
|
11
|
-
date: 2020-
|
11
|
+
date: 2020-09-28 00:00:00.000000000 Z
|
12
12
|
dependencies:
|
13
13
|
- !ruby/object:Gem::Dependency
|
14
14
|
name: sequel
|
15
15
|
requirement: !ruby/object:Gem::Requirement
|
16
16
|
requirements:
|
17
|
-
- - "
|
18
|
-
- !ruby/object:Gem::Version
|
19
|
-
version: '4.0'
|
20
|
-
- - "<"
|
17
|
+
- - "~>"
|
21
18
|
- !ruby/object:Gem::Version
|
22
|
-
version: '
|
19
|
+
version: '5.16'
|
23
20
|
type: :runtime
|
24
21
|
prerelease: false
|
25
22
|
version_requirements: !ruby/object:Gem::Requirement
|
26
23
|
requirements:
|
27
|
-
- - "
|
28
|
-
- !ruby/object:Gem::Version
|
29
|
-
version: '4.0'
|
30
|
-
- - "<"
|
24
|
+
- - "~>"
|
31
25
|
- !ruby/object:Gem::Version
|
32
|
-
version: '
|
26
|
+
version: '5.16'
|
33
27
|
- !ruby/object:Gem::Dependency
|
34
28
|
name: activerecord
|
35
29
|
requirement: !ruby/object:Gem::Requirement
|