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);