You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by om...@apache.org on 2007/10/15 20:59:17 UTC

svn commit: r584851 - in /lucene/hadoop/branches/branch-0.15: CHANGES.txt src/java/org/apache/hadoop/mapred/BasicTypeSorterBase.java

Author: omalley
Date: Mon Oct 15 11:59:17 2007
New Revision: 584851

URL: http://svn.apache.org/viewvc?rev=584851&view=rev
Log:
Merge -r 584823:58424 from trunk to branch 15 to fix HADOOP-2053.

Modified:
    lucene/hadoop/branches/branch-0.15/CHANGES.txt
    lucene/hadoop/branches/branch-0.15/src/java/org/apache/hadoop/mapred/BasicTypeSorterBase.java

Modified: lucene/hadoop/branches/branch-0.15/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/hadoop/branches/branch-0.15/CHANGES.txt?rev=584851&r1=584850&r2=584851&view=diff
==============================================================================
--- lucene/hadoop/branches/branch-0.15/CHANGES.txt (original)
+++ lucene/hadoop/branches/branch-0.15/CHANGES.txt Mon Oct 15 11:59:17 2007
@@ -415,6 +415,12 @@
     edits log. Reduce the number of syncs by double-buffering the changes
     to the transaction log. (Dhruba Borthakur)
 
+Release 0.14.3 - Unreleased
+
+  BUG FIXES
+
+    HADOOP-2053. Fixed a dangling reference to a memory buffer in the map 
+    output sorter. (acmurthy via omalley)
 
     HADOOP-2036. Fix a NullPointerException in JvmMetrics class. (nigel)
 

Modified: lucene/hadoop/branches/branch-0.15/src/java/org/apache/hadoop/mapred/BasicTypeSorterBase.java
URL: http://svn.apache.org/viewvc/lucene/hadoop/branches/branch-0.15/src/java/org/apache/hadoop/mapred/BasicTypeSorterBase.java?rev=584851&r1=584850&r2=584851&view=diff
==============================================================================
--- lucene/hadoop/branches/branch-0.15/src/java/org/apache/hadoop/mapred/BasicTypeSorterBase.java (original)
+++ lucene/hadoop/branches/branch-0.15/src/java/org/apache/hadoop/mapred/BasicTypeSorterBase.java Mon Oct 15 11:59:17 2007
@@ -120,6 +120,10 @@
     pointers = null;
     maxKeyLength = 0;
     maxValLength = 0;
+    
+    //release the large key-value buffer so that the GC, if necessary,
+    //can collect it away
+    keyValBuffer = null;
   }
   //A compare method that references the keyValBuffer through the indirect
   //pointers