Fixed bug and adding unit test for removeBroadcast in BlockManagerSuite.

witgo · Apr 4, 2014 · 6222697 · 6222697
1 parent 104a89a
commit 6222697
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/core/src/main/scala/org/apache/spark/storage/ShuffleBlockManager.scala b/core/src/main/scala/org/apache/spark/storage/ShuffleBlockManager.scala
@@ -170,9 +170,9 @@ class ShuffleBlockManager(blockManager: BlockManager) extends Logging {
   }
 
   /** Remove all the blocks / files and metadata related to a particular shuffle. */
-  def removeShuffle(shuffleId: ShuffleId) {
-    removeShuffleBlocks(shuffleId)
+  def removeShuffle(shuffleId: ShuffleId): Boolean = {
     shuffleStates.remove(shuffleId)
+    removeShuffleBlocks(shuffleId)
   }
 
   /** Remove all the blocks / files related to a particular shuffle. */