Repository: spark
Updated Branches:
  refs/heads/master 7620aed82 -> 650d03cfc


[SPARK-19998][BLOCK MANAGER] Change the exception log to add RDD id of the 
related the block

## What changes were proposed in this pull request?

"java.lang.Exception: Could not compute split, block $blockId not found" 
doesn't have the rdd id info, the "BlockManager: Removing RDD $id" has only the 
RDD id, so it couldn't find that the Exception's reason is the Removing; so 
it's better block not found Exception add RDD id info

## How was this patch tested?

Existing tests

Author: jianran.tfh <jianran....@taobao.com>
Author: jianran <tanfanhua1...@163.com>

Closes #17334 from jianran/SPARK-19998.


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

Branch: refs/heads/master
Commit: 650d03cfc9a609a2c603f9ced452d03ec8429b0d
Parents: 7620aed
Author: jianran.tfh <jianran....@taobao.com>
Authored: Tue Mar 21 15:15:19 2017 +0000
Committer: Sean Owen <so...@cloudera.com>
Committed: Tue Mar 21 15:15:19 2017 +0000

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


http://git-wip-us.apache.org/repos/asf/spark/blob/650d03cf/core/src/main/scala/org/apache/spark/rdd/BlockRDD.scala
----------------------------------------------------------------------
diff --git a/core/src/main/scala/org/apache/spark/rdd/BlockRDD.scala 
b/core/src/main/scala/org/apache/spark/rdd/BlockRDD.scala
index d47b755..4e036c2 100644
--- a/core/src/main/scala/org/apache/spark/rdd/BlockRDD.scala
+++ b/core/src/main/scala/org/apache/spark/rdd/BlockRDD.scala
@@ -47,7 +47,7 @@ class BlockRDD[T: ClassTag](sc: SparkContext, @transient val 
blockIds: Array[Blo
     blockManager.get[T](blockId) match {
       case Some(block) => block.data.asInstanceOf[Iterator[T]]
       case None =>
-        throw new Exception("Could not compute split, block " + blockId + " 
not found")
+        throw new Exception(s"Could not compute split, block $blockId of RDD 
$id not found")
     }
   }
 


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

Reply via email to