You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@chukwa.apache.org by as...@apache.org on 2009/03/27 18:49:00 UTC

svn commit: r759286 - in /hadoop/chukwa/trunk: CHANGES.txt src/java/org/apache/hadoop/chukwa/extraction/database/MetricDataLoader.java

Author: asrabkin
Date: Fri Mar 27 17:48:59 2009
New Revision: 759286

URL: http://svn.apache.org/viewvc?rev=759286&view=rev
Log:
CHUKWA-9.  MetricDataLoader should close JDBC connection.  (jboulon via asrabkin)

Modified:
    hadoop/chukwa/trunk/CHANGES.txt
    hadoop/chukwa/trunk/src/java/org/apache/hadoop/chukwa/extraction/database/MetricDataLoader.java

Modified: hadoop/chukwa/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/chukwa/trunk/CHANGES.txt?rev=759286&r1=759285&r2=759286&view=diff
==============================================================================
--- hadoop/chukwa/trunk/CHANGES.txt (original)
+++ hadoop/chukwa/trunk/CHANGES.txt Fri Mar 27 17:48:59 2009
@@ -35,6 +35,8 @@
 
   BUG FIXES
 
+    CHUKWA-9.  MetricDataLoader should close JDBC connection.  (Jerome Boulon via asrabkin)
+
     CHUKWA-37.  Remove ChuckwaArchiveBuilder. (asrabkin)
 
     CHUKWA-33.  Reformat code to fit hadoop style.  (asrabkin)

Modified: hadoop/chukwa/trunk/src/java/org/apache/hadoop/chukwa/extraction/database/MetricDataLoader.java
URL: http://svn.apache.org/viewvc/hadoop/chukwa/trunk/src/java/org/apache/hadoop/chukwa/extraction/database/MetricDataLoader.java?rev=759286&r1=759285&r2=759286&view=diff
==============================================================================
--- hadoop/chukwa/trunk/src/java/org/apache/hadoop/chukwa/extraction/database/MetricDataLoader.java (original)
+++ hadoop/chukwa/trunk/src/java/org/apache/hadoop/chukwa/extraction/database/MetricDataLoader.java Fri Mar 27 17:48:59 2009
@@ -366,6 +366,15 @@
         }
         stmt = null;
       }
+      if (conn != null) {
+        try {
+          conn.close();
+        } catch (SQLException sqlEx) {
+          // ignore
+        }
+        conn = null;
+      }
+
     }
   }