Repository: spark
Updated Branches:
  refs/heads/master 72df5a301 -> a00af6bec


[SPARK-4973][CORE] Local directory in the driver of client-mode continues 
remaining even if application finished when external shuffle is enabled

When we enables external shuffle service, local directories in the driver of 
client-mode continue remaining even if application has finished.
I think local directories for drivers should be deleted.

Author: Kousuke Saruta <saru...@oss.nttdata.co.jp>

Closes #3811 from sarutak/SPARK-4973 and squashes the following commits:

ad944ab [Kousuke Saruta] Fixed DiskBlockManager to cleanup local directory if 
it's the driver
43770da [Kousuke Saruta] Merge branch 'master' of git://git.apache.org/spark 
into SPARK-4973
88feecd [Kousuke Saruta] Merge branch 'master' of git://git.apache.org/spark 
into SPARK-4973
d99718e [Kousuke Saruta] Fixed SparkSubmit.scala and DiskBlockManager.scala in 
order to delete local directories of the driver of local-mode when external 
shuffle service is enabled


Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/a00af6be
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/a00af6be
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/a00af6be

Branch: refs/heads/master
Commit: a00af6bec57b8df8b286aaa5897232475aef441c
Parents: 72df5a3
Author: Kousuke Saruta <saru...@oss.nttdata.co.jp>
Authored: Thu Jan 8 13:43:09 2015 -0800
Committer: Andrew Or <and...@databricks.com>
Committed: Thu Jan 8 13:43:09 2015 -0800

----------------------------------------------------------------------
 .../src/main/scala/org/apache/spark/storage/DiskBlockManager.scala | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/a00af6be/core/src/main/scala/org/apache/spark/storage/DiskBlockManager.scala
----------------------------------------------------------------------
diff --git 
a/core/src/main/scala/org/apache/spark/storage/DiskBlockManager.scala 
b/core/src/main/scala/org/apache/spark/storage/DiskBlockManager.scala
index bb2ae9f..af05eb3 100644
--- a/core/src/main/scala/org/apache/spark/storage/DiskBlockManager.scala
+++ b/core/src/main/scala/org/apache/spark/storage/DiskBlockManager.scala
@@ -166,7 +166,7 @@ private[spark] class DiskBlockManager(blockManager: 
BlockManager, conf: SparkCon
   /** Cleanup local dirs and stop shuffle sender. */
   private[spark] def stop() {
     // Only perform cleanup if an external service is not serving our shuffle 
files.
-    if (!blockManager.externalShuffleServiceEnabled) {
+    if (!blockManager.externalShuffleServiceEnabled || 
blockManager.blockManagerId.isDriver) {
       localDirs.foreach { localDir =>
         if (localDir.isDirectory() && localDir.exists()) {
           try {


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to