commit 6607beab0dc8043251b490471761fa2dd85f2816
Author: Peter Portante <email address hidden>
Date: Fri Oct 4 08:04:42 2013 -0400
Don't apply timeout to Pool.get operation (leaks)
The connection timeout to a memcache server is performed by using the
"with Timeout()" construct over the sock.connect() call in the
.create() method. In addition, the same construct was being applied to
the Pool.get() call in ._get_conns().
If the maximum number of connections was already created, and the
Pool.get() called took longer than the connect timeout, then the error
handling path would add a place holder to the connection
pool. Eventlet's Pool class allows for additional items to be added to
the pool, over and above the max_size setting. This additional place
holder will eventually be pulled and a new connection created to take
its place.
The fix is to remove the timeout construct in the _get_conns() method.
Reviewed: https:/ /review. openstack. org/49739 github. com/openstack/ swift/commit/ 6607beab0dc8043 251b490471761fa 2dd85f2816
Committed: http://
Submitter: Jenkins
Branch: master
commit 6607beab0dc8043 251b490471761fa 2dd85f2816
Author: Peter Portante <email address hidden>
Date: Fri Oct 4 08:04:42 2013 -0400
Don't apply timeout to Pool.get operation (leaks)
The connection timeout to a memcache server is performed by using the
"with Timeout()" construct over the sock.connect() call in the
.create() method. In addition, the same construct was being applied to
the Pool.get() call in ._get_conns().
If the maximum number of connections was already created, and the
Pool.get() called took longer than the connect timeout, then the error
handling path would add a place holder to the connection
pool. Eventlet's Pool class allows for additional items to be added to
the pool, over and above the max_size setting. This additional place
holder will eventually be pulled and a new connection created to take
its place.
The fix is to remove the timeout construct in the _get_conns() method.
In addition, we also apply the unit test patch mentioned in the review /review. openstack. org/45134, paste.openstack .org/show/ 47288/.
comments for Patch Set 6 of https:/
located at http://
Fixes bug 1235027
Change-Id: I786cabefe3e8dd f7d92feb7ebc7cf b613d60a1da
Signed-off-by: Peter Portante <email address hidden>