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:52:32 UTC
svn commit: r1735212 -
/httpcomponents/httpcore/trunk/httpcore5/src/main/java/org/apache/hc/core5/pool/io/AbstractConnPool.java
Author: olegk
Date: Wed Mar 16 10:52:32 2016
New Revision: 1735212
URL: http://svn.apache.org/viewvc?rev=1735212&view=rev
Log:
Asymetric number of invocations of onLease/onRelease in conn pool
Modified:
httpcomponents/httpcore/trunk/httpcore5/src/main/java/org/apache/hc/core5/pool/io/AbstractConnPool.java
Modified: httpcomponents/httpcore/trunk/httpcore5/src/main/java/org/apache/hc/core5/pool/io/AbstractConnPool.java
URL: http://svn.apache.org/viewvc/httpcomponents/httpcore/trunk/httpcore5/src/main/java/org/apache/hc/core5/pool/io/AbstractConnPool.java?rev=1735212&r1=1735211&r2=1735212&view=diff
==============================================================================
--- httpcomponents/httpcore/trunk/httpcore5/src/main/java/org/apache/hc/core5/pool/io/AbstractConnPool.java (original)
+++ httpcomponents/httpcore/trunk/httpcore5/src/main/java/org/apache/hc/core5/pool/io/AbstractConnPool.java Wed Mar 16 10:52:32 2016
@@ -338,10 +338,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);