Repository: spark
Updated Branches:
  refs/heads/master dedace83f -> e678b9f02


[SPARK-3973] Print call site information for broadcasts

Its hard to debug which broadcast variables refer to what in a big codebase. 
Printing call site information helps in debugging.

Author: Shivaram Venkataraman <[email protected]>

Closes #2829 from shivaram/spark-broadcast-print and squashes the following 
commits:

cd6dbdf [Shivaram Venkataraman] Print call site information for broadcasts


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

Branch: refs/heads/master
Commit: e678b9f02a2936b35c95e91a5f0ff388b5720261
Parents: dedace8
Author: Shivaram Venkataraman <[email protected]>
Authored: Thu Oct 16 19:43:33 2014 -0700
Committer: Josh Rosen <[email protected]>
Committed: Thu Oct 16 19:43:33 2014 -0700

----------------------------------------------------------------------
 core/src/main/scala/org/apache/spark/SparkContext.scala | 2 ++
 1 file changed, 2 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/e678b9f0/core/src/main/scala/org/apache/spark/SparkContext.scala
----------------------------------------------------------------------
diff --git a/core/src/main/scala/org/apache/spark/SparkContext.scala 
b/core/src/main/scala/org/apache/spark/SparkContext.scala
index 3541162..dd31579 100644
--- a/core/src/main/scala/org/apache/spark/SparkContext.scala
+++ b/core/src/main/scala/org/apache/spark/SparkContext.scala
@@ -818,6 +818,8 @@ class SparkContext(config: SparkConf) extends Logging {
    */
   def broadcast[T: ClassTag](value: T): Broadcast[T] = {
     val bc = env.broadcastManager.newBroadcast[T](value, isLocal)
+    val callSite = getCallSite
+    logInfo("Created broadcast " + bc.id + " from " + callSite.shortForm)
     cleaner.foreach(_.registerBroadcastForCleanup(bc))
     bc
   }


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to