diff options
author | Matthew Draper <matthew@trebex.net> | 2016-07-07 07:40:26 +0930 |
---|---|---|
committer | GitHub <noreply@github.com> | 2016-07-07 07:40:26 +0930 |
commit | aeba05d389a846908cedbafa3a82eb781ae0f19d (patch) | |
tree | 51538a6aabc93247cef3da4bb0d70a47014ae5ec /activerecord/lib/active_record/connection_adapters/abstract/connection_pool.rb | |
parent | d5ca0a8b8e4a9ab301ba37a335e62ada872e7a76 (diff) | |
parent | 61f4b1ff8a54ccd7e70bb4f6c0b958b28bc648fc (diff) | |
download | rails-aeba05d389a846908cedbafa3a82eb781ae0f19d.tar.gz rails-aeba05d389a846908cedbafa3a82eb781ae0f19d.tar.bz2 rails-aeba05d389a846908cedbafa3a82eb781ae0f19d.zip |
Merge pull request #25707 from matthewd/double-reap
Don't reap connections that have already been reassigned
Diffstat (limited to 'activerecord/lib/active_record/connection_adapters/abstract/connection_pool.rb')
-rw-r--r-- | activerecord/lib/active_record/connection_adapters/abstract/connection_pool.rb | 24 |
1 files changed, 15 insertions, 9 deletions
diff --git a/activerecord/lib/active_record/connection_adapters/abstract/connection_pool.rb b/activerecord/lib/active_record/connection_adapters/abstract/connection_pool.rb index a006e4f50f..9b74c3a10f 100644 --- a/activerecord/lib/active_record/connection_adapters/abstract/connection_pool.rb +++ b/activerecord/lib/active_record/connection_adapters/abstract/connection_pool.rb @@ -415,7 +415,10 @@ module ActiveRecord with_exclusively_acquired_all_connections(raise_on_acquisition_timeout) do synchronize do @connections.each do |conn| - checkin conn + if conn.in_use? + conn.steal! + checkin conn + end conn.disconnect! end @connections = [] @@ -447,7 +450,10 @@ module ActiveRecord with_exclusively_acquired_all_connections(raise_on_acquisition_timeout) do synchronize do @connections.each do |conn| - checkin conn + if conn.in_use? + conn.steal! + checkin conn + end conn.disconnect! if conn.requires_reloading? end @connections.delete_if(&:requires_reloading?) @@ -556,17 +562,17 @@ module ActiveRecord stale_connections = synchronize do @connections.select do |conn| conn.in_use? && !conn.owner.alive? + end.each do |conn| + conn.steal! end end stale_connections.each do |conn| - synchronize do - if conn.active? - conn.reset! - checkin conn - else - remove conn - end + if conn.active? + conn.reset! + checkin conn + else + remove conn end end end |