Browse files

make sure connections returned after close are marked as in_use

  • Loading branch information...
1 parent da0595d commit 5f26ce699f9e695c434cbff20626a9ff4d3114e4 @tenderlove tenderlove committed Mar 12, 2012
View
16 activerecord/lib/active_record/connection_adapters/abstract/connection_pool.rb
@@ -269,11 +269,27 @@ def checkin(conn)
conn.expire
@queue.signal
end
+
+ release conn
end
end
private
+ def release(conn)
+ thread_id = nil
+
+ if @reserved_connections[current_connection_id] == conn
+ thread_id = current_connection_id
+ else
+ thread_id = @reserved_connections.keys.find { |k|
+ @reserved_connections[k] == conn
+ }
+ end
+
+ @reserved_connections.delete thread_id if thread_id
+ end
+
def new_connection
ActiveRecord::Base.send(spec.adapter_method, spec.config)
end
View
10 activerecord/test/cases/connection_pool_test.rb
@@ -31,6 +31,16 @@ def active_connections(pool)
pool.connections.find_all(&:in_use?)
end
+ def test_checkout_after_close
+ connection = pool.connection
+ assert connection.in_use?
+
+ connection.close
+ assert !connection.in_use?
+
+ assert pool.connection.in_use?
+ end
+
def test_released_connection_moves_between_threads
thread_conn = nil

0 comments on commit 5f26ce6

Please sign in to comment.