From 942847fd94c920f7954ddf01f97263926e512b0e Mon Sep 17 00:00:00 2001
From: omgteam <Kimlong.Liu@gmail.com>
Date: Mon, 13 Oct 2014 09:59:41 -0700
Subject: [PATCH] Bug Fix: without unpersist method in RandomForest.scala
During trainning Gradient Boosting Decision Tree on large-scale sparse data, spark spill hundreds of data onto disk. And find the bug below:
In version 1.1.0 DecisionTree.scala, train Method, treeInput has been persisted in Memory, but without unpersist. It caused heavy DISK usage.
In github version(1.2.0 maybe), RandomForest.scala, train Method, baggedInput has been persisted but without unpersisted too.
After added unpersist, it works right.
https://issues.apache.org/jira/browse/SPARK-3918
Author: omgteam <Kimlong.Liu@gmail.com>
Closes #2775 from omgteam/master and squashes the following commits:
815d543 [omgteam] adjust tab to spaces
1a36f83 [omgteam] Bug: fix without unpersist baggedInput in RandomForest.scala
---
.../main/scala/org/apache/spark/mllib/tree/RandomForest.scala | 2 ++
1 file changed, 2 insertions(+)
diff --git a/mllib/src/main/scala/org/apache/spark/mllib/tree/RandomForest.scala b/mllib/src/main/scala/org/apache/spark/mllib/tree/RandomForest.scala
index fa7a26f17c..ebbd8e0257 100644
--- a/mllib/src/main/scala/org/apache/spark/mllib/tree/RandomForest.scala
+++ b/mllib/src/main/scala/org/apache/spark/mllib/tree/RandomForest.scala
@@ -176,6 +176,8 @@ private class RandomForest (
timer.stop("findBestSplits")
}
+ baggedInput.unpersist()
+
timer.stop("total")
logInfo("Internal timing for DecisionTree:")
--
GitLab