SPARK-11246 implement code review comment

xwu0226 · xwu0226 · commit 88edd14e6cb6 · 2015-10-28T22:37:33.000-07:00
diff --git a/sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/LogicalRelation.scala b/sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/LogicalRelation.scala
@@ -63,8 +63,8 @@ case class LogicalRelation(
   }
 
   // When comparing two LogicalRelations from within LogicalPlan.sameResult, we only need
-  // LogicalRelation.cleanArgs to return Seq(relation), since expectedOutputAttribute's expId can be different
-  // but the relation is still the same.
+  // LogicalRelation.cleanArgs to return Seq(relation), since expectedOutputAttribute's
+  // expId can be different but the relation is still the same.
   override lazy val cleanArgs: Seq[Any] = Seq(relation)
 
   @transient override lazy val statistics: Statistics = Statistics(

Original file line number	Diff line number	Diff line change
`@@ -63,8 +63,8 @@ case class LogicalRelation(`
`63`	`63`	`}`
`64`	`64`
`65`	`65`	`// When comparing two LogicalRelations from within LogicalPlan.sameResult, we only need`
`66`		`- // LogicalRelation.cleanArgs to return Seq(relation), since expectedOutputAttribute's expId can be different`
`67`		`- // but the relation is still the same.`
	`66`	`+ // LogicalRelation.cleanArgs to return Seq(relation), since expectedOutputAttribute's`
	`67`	`+ // expId can be different but the relation is still the same.`
`68`	`68`	`override lazy val cleanArgs: Seq[Any] = Seq(relation)`
`69`	`69`
`70`	`70`	`@transient override lazy val statistics: Statistics = Statistics(`