Add comment.

apache · Jul 29, 2014 · 3ba8f3e · 3ba8f3e
1 parent e5bcf5b
commit 3ba8f3e
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/sql/core/src/main/scala/org/apache/spark/sql/execution/SparkPlan.scala b/sql/core/src/main/scala/org/apache/spark/sql/execution/SparkPlan.scala
@@ -87,6 +87,8 @@ case class SparkLogicalPlan(alreadyPlanned: SparkPlan)
     override lazy val sizeInBytes: Long = {
       val defaultSum = childrenStats.map(_.sizeInBytes).sum
       alreadyPlanned match {
+        // TODO: Instead of returning a default value here, find a way to return a meaningful
+        // estimate for RDDs. See PR 1238 for more discussions.
         case e: ExistingRdd if defaultSum == 0 => statsDefaultSizeInBytes
         case _ => defaultSum
       }