review comments

dilipbiswal · dilipbiswal · commit af3d3674b057 · 2017-04-17T14:55:03.000-07:00
diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
@@ -1240,7 +1240,7 @@ class Analyzer(
       }
 
       // Make sure a plan's expressions do not contain :
-      // 1. Aggregate expressions that has mixture of outer and local references.
+      // 1. Aggregate expressions that have mixture of outer and local references.
       // 2. Expressions containing outer references on plan nodes other than Filter.
       def failOnInvalidOuterReference(p: LogicalPlan): Unit = {
         p.expressions.foreach(checkMixedReferencesInsideAggregateExpr)

Original file line number	Diff line number	Diff line change
`@@ -1240,7 +1240,7 @@ class Analyzer(`
`1240`	`1240`	`}`
`1241`	`1241`
`1242`	`1242`	`// Make sure a plan's expressions do not contain :`
`1243`		`- // 1. Aggregate expressions that has mixture of outer and local references.`
	`1243`	`+ // 1. Aggregate expressions that have mixture of outer and local references.`
`1244`	`1244`	`// 2. Expressions containing outer references on plan nodes other than Filter.`
`1245`	`1245`	`def failOnInvalidOuterReference(p: LogicalPlan): Unit = {`
`1246`	`1246`	`p.expressions.foreach(checkMixedReferencesInsideAggregateExpr)`