You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by nz...@apache.org on 2011/03/17 20:25:53 UTC

svn commit: r1082652 - /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistory.java

Author: nzhang
Date: Thu Mar 17 19:25:53 2011
New Revision: 1082652

URL: http://svn.apache.org/viewvc?rev=1082652&view=rev
Log:
HIVE-1959. Potential memory leak when same connection used for long time. TaskInfo and QueryInfo objects are getting accumulated on executing more queries on the same connection. (Chinna Rao Lalam via Ning Zhang)

Modified:
    hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistory.java

Modified: hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistory.java
URL: http://svn.apache.org/viewvc/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistory.java?rev=1082652&r1=1082651&r2=1082652&view=diff
==============================================================================
--- hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistory.java (original)
+++ hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistory.java Thu Mar 17 19:25:53 2011
@@ -425,6 +425,7 @@ public class HiveHistory {
       return;
     }
     log(RecordTypes.QueryEnd, ji.hm);
+    queryInfoMap.remove(queryId);
   }
 
   /**
@@ -465,6 +466,7 @@ public class HiveHistory {
       return;
     }
     log(RecordTypes.TaskEnd, ti.hm);
+    taskInfoMap.remove(id);
   }
 
   /**