Author: markt Date: Wed Apr 1 14:58:47 2009 New Revision: 760907 URL: http://svn.apache.org/viewvc?rev=760907&view=rev Log: Fix POOL-122. Given that stdout and stderr are the only options available for logging, don't log any more or less than previously but give the evictor a chance to carry on.
Modified: commons/proper/pool/trunk/src/java/org/apache/commons/pool/impl/GenericKeyedObjectPool.java commons/proper/pool/trunk/src/java/org/apache/commons/pool/impl/GenericObjectPool.java Modified: commons/proper/pool/trunk/src/java/org/apache/commons/pool/impl/GenericKeyedObjectPool.java URL: http://svn.apache.org/viewvc/commons/proper/pool/trunk/src/java/org/apache/commons/pool/impl/GenericKeyedObjectPool.java?rev=760907&r1=760906&r2=760907&view=diff ============================================================================== --- commons/proper/pool/trunk/src/java/org/apache/commons/pool/impl/GenericKeyedObjectPool.java (original) +++ commons/proper/pool/trunk/src/java/org/apache/commons/pool/impl/GenericKeyedObjectPool.java Wed Apr 1 14:58:47 2009 @@ -1759,6 +1759,10 @@ evict(); } catch(Exception e) { // ignored + } catch(Throwable t) { + // Log problem but give evictor thread a chance to continue in + // case error is recoverable + t.printStackTrace(System.err); } //Re-create the connections. try { Modified: commons/proper/pool/trunk/src/java/org/apache/commons/pool/impl/GenericObjectPool.java URL: http://svn.apache.org/viewvc/commons/proper/pool/trunk/src/java/org/apache/commons/pool/impl/GenericObjectPool.java?rev=760907&r1=760906&r2=760907&view=diff ============================================================================== --- commons/proper/pool/trunk/src/java/org/apache/commons/pool/impl/GenericObjectPool.java (original) +++ commons/proper/pool/trunk/src/java/org/apache/commons/pool/impl/GenericObjectPool.java Wed Apr 1 14:58:47 2009 @@ -1348,6 +1348,10 @@ evict(); } catch(Exception e) { // ignored + } catch(Throwable t) { + // Log problem but give evictor thread a chance to continue in + // case error is recoverable + t.printStackTrace(System.err); } try { ensureMinIdle();