You are viewing a plain text version of this content. The canonical link for it is here.

Posted to mapreduce-commits@hadoop.apache.org by to...@apache.org on 2009/08/12 16:37:26 UTC

svn commit: r803537 - in /hadoop/mapreduce/trunk: CHANGES.txt src/java/org/apache/hadoop/mapreduce/lib/db/DBInputFormat.java

Author: tomwhite
Date: Wed Aug 12 14:37:26 2009
New Revision: 803537

URL: http://svn.apache.org/viewvc?rev=803537&view=rev
Log:
MAPREDUCE-840. DBInputFormat leaves open transaction. Contributed by Aaron Kimball.

Modified:
    hadoop/mapreduce/trunk/CHANGES.txt
    hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapreduce/lib/db/DBInputFormat.java

Modified: hadoop/mapreduce/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/mapreduce/trunk/CHANGES.txt?rev=803537&r1=803536&r2=803537&view=diff
==============================================================================
--- hadoop/mapreduce/trunk/CHANGES.txt (original)
+++ hadoop/mapreduce/trunk/CHANGES.txt Wed Aug 12 14:37:26 2009
@@ -362,3 +362,6 @@
 
     MAPREDUCE-848. Fixes a problem to do with TestCapacityScheduler
     failing (Amar Kamat via ddas)
+
+    MAPREDUCE-840. DBInputFormat leaves open transaction.
+    (Aaron Kimball via tomwhite)

Modified: hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapreduce/lib/db/DBInputFormat.java
URL: http://svn.apache.org/viewvc/hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapreduce/lib/db/DBInputFormat.java?rev=803537&r1=803536&r2=803537&view=diff
==============================================================================
--- hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapreduce/lib/db/DBInputFormat.java (original)
+++ hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapreduce/lib/db/DBInputFormat.java Wed Aug 12 14:37:26 2009
@@ -240,6 +240,7 @@
         splits.add(split);
       }
 
+      connection.commit();
       return splits;
     } catch (SQLException e) {
       try {