Skip to content

Commit

Permalink
Fix actual bug.
Browse files Browse the repository at this point in the history
  • Loading branch information
JoshRosen committed Jun 2, 2016
1 parent a104ac5 commit e26b2f6
Showing 1 changed file with 6 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -442,6 +442,8 @@ private[spark] class BlockManager(
Some(new BlockResult(ci, DataReadMethod.Disk, info.size))
} else {
releaseLock(blockId)
// Remove the missing block so that its unavailability is reported to the driver
removeBlock(blockId)
throw new SparkException(s"Block $blockId was not found even though it's read-locked")
}
}
Expand Down Expand Up @@ -490,6 +492,8 @@ private[spark] class BlockManager(
serializerManager.dataSerialize(blockId, memoryStore.getValues(blockId).get)
} else {
releaseLock(blockId)
// Remove the missing block so that its unavailability is reported to the driver
removeBlock(blockId)
throw new SparkException(s"Block $blockId was not found even though it's read-locked")
}
} else { // storage level is serialized
Expand All @@ -500,6 +504,8 @@ private[spark] class BlockManager(
maybeCacheDiskBytesInMemory(info, blockId, level, diskBytes).getOrElse(diskBytes)
} else {
releaseLock(blockId)
// Remove the missing block so that its unavailability is reported to the driver
removeBlock(blockId)
throw new SparkException(s"Block $blockId was not found even though it's read-locked")
}
}
Expand Down

0 comments on commit e26b2f6

Please sign in to comment.