You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hc.apache.org by ol...@apache.org on 2016/03/16 11:39:09 UTC
svn commit: r1735209 -
/httpcomponents/httpcore/branches/4.4.x/httpcore/src/main/java/org/apache/http/pool/AbstractConnPool.java
Author: olegk
Date: Wed Mar 16 10:39:09 2016
New Revision: 1735209
URL: http://svn.apache.org/viewvc?rev=1735209&view=rev
Log:
Asymetric number of invocations of onLease/onRelease in conn pool
Modified:
httpcomponents/httpcore/branches/4.4.x/httpcore/src/main/java/org/apache/http/pool/AbstractConnPool.java
Modified: httpcomponents/httpcore/branches/4.4.x/httpcore/src/main/java/org/apache/http/pool/AbstractConnPool.java
URL: http://svn.apache.org/viewvc/httpcomponents/httpcore/branches/4.4.x/httpcore/src/main/java/org/apache/http/pool/AbstractConnPool.java?rev=1735209&r1=1735208&r2=1735209&view=diff
==============================================================================
--- httpcomponents/httpcore/branches/4.4.x/httpcore/src/main/java/org/apache/http/pool/AbstractConnPool.java (original)
+++ httpcomponents/httpcore/branches/4.4.x/httpcore/src/main/java/org/apache/http/pool/AbstractConnPool.java Wed Mar 16 10:39:09 2016
@@ -333,10 +333,10 @@ public abstract class AbstractConnPool<T
pool.free(entry, reusable);
if (reusable && !this.isShutDown) {
this.available.addFirst(entry);
- onRelease(entry);
} else {
entry.close();
}
+ onRelease(entry);
PoolEntryFuture<E> future = pool.nextPending();
if (future != null) {
this.pending.remove(future);