commons-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ma...@apache.org
Subject svn commit: r760907 - in /commons/proper/pool/trunk/src/java/org/apache/commons/pool/impl: GenericKeyedObjectPool.java GenericObjectPool.java
Date Wed, 01 Apr 2009 14:58:51 GMT
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();



Mime
View raw message