On 01/05/2012 10:44, sebb wrote: > On 1 May 2012 10:25, <ma...@apache.org> wrote: >> Author: markt >> Date: Tue May 1 09:25:35 2012 >> New Revision: 1332588 >> >> URL: http://svn.apache.org/viewvc?rev=1332588&view=rev >> Log: >> Refactor so the Javadoc for GKOP doesn't point to GOP. > > The Javadoc originally referred to a missing method, so I changed it > to use an interface of BGOP, which is inherited by GOP and GKOP. > > Not sure how that can have resulted in GKOP referencing GOP? I don't > see that at all. > > I'm only curious about the log comment; the actual change is fine.
That is what I get for (mis) reading the diff on my phone. Mark > >> Modified: >> >> commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/impl/BaseGenericObjectPool.java >> >> commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/impl/GenericKeyedObjectPool.java >> >> commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/impl/GenericObjectPool.java >> >> Modified: >> commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/impl/BaseGenericObjectPool.java >> URL: >> http://svn.apache.org/viewvc/commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/impl/BaseGenericObjectPool.java?rev=1332588&r1=1332587&r2=1332588&view=diff >> ============================================================================== >> --- >> commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/impl/BaseGenericObjectPool.java >> (original) >> +++ >> commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/impl/BaseGenericObjectPool.java >> Tue May 1 09:25:35 2012 >> @@ -397,7 +397,7 @@ public abstract class BaseGenericObjectP >> * of the idle object evictor thread. When positive, the number of tests >> * performed for a run will be the minimum of the configured value and >> the >> * number of idle instances in the pool. When negative, the number of >> tests >> - * performed will be <code>ceil({@link >> org.apache.commons.pool2.ObjectPool#getNumIdle #getNumIdle}/ >> + * performed will be <code>ceil({@link #getNumIdle}/ >> * abs({@link #getNumTestsPerEvictionRun})) whch means that when the >> value >> * is <code>-n</code> roughly one nth of the idle objects will be tested >> per >> * run. >> @@ -416,7 +416,7 @@ public abstract class BaseGenericObjectP >> * of the idle object evictor thread. When positive, the number of tests >> * performed for a run will be the minimum of the configured value and >> the >> * number of idle instances in the pool. When negative, the number of >> tests >> - * performed will be <code>ceil({@link >> org.apache.commons.pool2.ObjectPool#getNumIdle() #getNumIdle()}/ >> + * performed will be <code>ceil({@link #getNumIdle}/ >> * abs({@link #getNumTestsPerEvictionRun})) whch means that when the >> value >> * is <code>-n</code> roughly one nth of the idle objects will be tested >> per >> * run. >> @@ -737,6 +737,11 @@ public abstract class BaseGenericObjectP >> return maxBorrowWaitTimeMillis; >> } >> >> + /** >> + * The number of instances currently idle in this pool. >> + */ >> + public abstract int getNumIdle(); >> + >> final NotificationBroadcasterSupport getJmxNotificationSupport() { >> return jmxNotificationSupport; >> } >> >> Modified: >> commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/impl/GenericKeyedObjectPool.java >> URL: >> http://svn.apache.org/viewvc/commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/impl/GenericKeyedObjectPool.java?rev=1332588&r1=1332587&r2=1332588&view=diff >> ============================================================================== >> --- >> commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/impl/GenericKeyedObjectPool.java >> (original) >> +++ >> commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/impl/GenericKeyedObjectPool.java >> Tue May 1 09:25:35 2012 >> @@ -743,11 +743,6 @@ public class GenericKeyedObjectPool<K,T> >> return numTotal.get() - getNumIdle(); >> } >> >> - /** >> - * Returns the total number of instances currently idle in this pool. >> - * >> - * @return the total number of instances currently idle in this pool >> - */ >> @Override >> public int getNumIdle() { >> Iterator<ObjectDeque<T>> iter = poolMap.values().iterator(); >> >> Modified: >> commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/impl/GenericObjectPool.java >> URL: >> http://svn.apache.org/viewvc/commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/impl/GenericObjectPool.java?rev=1332588&r1=1332587&r2=1332588&view=diff >> ============================================================================== >> --- >> commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/impl/GenericObjectPool.java >> (original) >> +++ >> commons/proper/pool/trunk/src/main/java/org/apache/commons/pool2/impl/GenericObjectPool.java >> Tue May 1 09:25:35 2012 >> @@ -630,11 +630,6 @@ public class GenericObjectPool<T> extend >> return allObjects.size() - idleObjects.size(); >> } >> >> - /** >> - * Return the number of instances currently idle in this pool. >> - * >> - * @return the number of instances currently idle in this pool >> - */ >> @Override >> public int getNumIdle() { >> return idleObjects.size(); >> >> > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org