kotovalexarian-likes-github/rails--rails
1
0
Fork 0
mirror of https://github.com/rails/rails.git synced 2022-11-09 12:12:34 -05:00
Code Releases Activity

Merge pull request #27057 from kamipo/fix_race_condition

Browse source 
Fix the race condition caused by `with_new_connections_blocked`
This commit is contained in:
Matthew Draper 2016-11-19 20:24:59 +10:30
commit f9230a2d42
1 changed files with 7 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
activerecord/lib/active_record/connection_adapters/abstract/connection_pool.rb
View file

@ -350,8 +350,7 @@ module ActiveRecord
# currently in the process of independently establishing connections to the DB.
@now_connecting = 0
# A boolean toggle that allows/disallows new connections.
@new_cons_enabled = true
@threads_blocking_new_connections = 0
@available = ConnectionLeasingQueue.new self
end
@ -700,13 +699,15 @@ module ActiveRecord
end
def with_new_connections_blocked
previous_value = nil
synchronize do
previous_value, @new_cons_enabled = @new_cons_enabled, false
@threads_blocking_new_connections += 1
end
yield
ensure
synchronize { @new_cons_enabled = previous_value }
synchronize do
@threads_blocking_new_connections -= 1
end
end
# Acquire a connection by one of 1) immediately removing one
@ -758,7 +759,7 @@ module ActiveRecord
# and increment @now_connecting, to prevent overstepping this pool's @size
# constraint
do_checkout = synchronize do
if @new_cons_enabled && (@connections.size + @now_connecting) < @size
if @threads_blocking_new_connections.zero? && (@connections.size + @now_connecting) < @size
@now_connecting += 1
end
end