From bb3b68a18957f279926b2007de286f4c49d14184 Mon Sep 17 00:00:00 2001
From: wenshao <szujobs@hotmail.com>
Date: Fri, 1 Jul 2011 08:27:14 +0000
Subject: [PATCH] =?UTF-8?q?=E4=BD=BF=E7=94=A8log?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

git-svn-id: http://code.alibabatech.com/svn/druid/trunk@347 b9813039-fb51-4c41-a8b9-e21c2acb5095
---
 .../com/alibaba/druid/pool/DruidDataSource.java     | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/src/main/java/com/alibaba/druid/pool/DruidDataSource.java b/src/main/java/com/alibaba/druid/pool/DruidDataSource.java
index f05dea731d..ed2727b552 100644
--- a/src/main/java/com/alibaba/druid/pool/DruidDataSource.java
+++ b/src/main/java/com/alibaba/druid/pool/DruidDataSource.java
@@ -42,6 +42,8 @@ import javax.naming.StringRefAddr;
 
 import com.alibaba.druid.filter.Filter;
 import com.alibaba.druid.filter.stat.StatFilter;
+import com.alibaba.druid.logging.Log;
+import com.alibaba.druid.logging.LogFactory;
 import com.alibaba.druid.pool.vendor.MSSQLValidConnectionChecker;
 import com.alibaba.druid.pool.vendor.MySqlValidConnectionChecker;
 import com.alibaba.druid.pool.vendor.OracleValidConnectionChecker;
@@ -55,6 +57,7 @@ import com.alibaba.druid.util.JdbcUtils;
  * @author wenshao<szujobs@hotmail.com>
  */
 public class DruidDataSource extends DruidAbstractDataSource implements DruidDataSourceMBean, Referenceable {
+    private final static Log LOG = LogFactory.getLog(DruidDataSource.class);
 
     private static final Object                                                  PRESENT                     = new Object();
     private static final IdentityHashMap<DruidDataSource, Object>                instances                   = new IdentityHashMap<DruidDataSource, Object>();
@@ -393,7 +396,7 @@ public class DruidDataSource extends DruidAbstractDataSource implements DruidDat
                     connections[i] = null;
                     destroyCount++;
                 } catch (Exception ex) {
-                    printStackTrace(ex);
+                    LOG.warn("close connection error", ex);
                 }
             }
             count = 0;
@@ -575,7 +578,7 @@ public class DruidDataSource extends DruidAbstractDataSource implements DruidDat
                 } catch (InterruptedException e) {
                     break;
                 } catch (SQLException e) {
-                    printStackTrace(e);
+                    LOG.error("create connection error", e);
 
                     errorCount++;
 
@@ -587,9 +590,9 @@ public class DruidDataSource extends DruidAbstractDataSource implements DruidDat
                         }
                     }
                 } catch (RuntimeException e) {
-                    printStackTrace(e);
+                    LOG.error("create connection error", e);
                 } catch (Error e) {
-                    printStackTrace(e);
+                    LOG.error("create connection error", e);
                     break;
                 } finally {
                     lock.unlock();
@@ -649,7 +652,7 @@ public class DruidDataSource extends DruidAbstractDataSource implements DruidDat
                     try {
                         connection.close();
                     } catch (SQLException e) {
-                        printStackTrace(e);
+                        LOG.error("create connection error", e);
                     }
 
                     destroyCount++;