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


Reply via email to