extra case

marmbrus · Feb 12, 2015 · d29fbde · d29fbde
1 parent 1a797b4
commit d29fbde
Showing 1 changed file with 0 additions and 1 deletion.
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
@@ -111,7 +111,6 @@ class Analyzer(catalog: Catalog,
                     s"expression '${e.prettyString}' is neither present in the group by, " +
                     s"nor is it an aggregate function. " +
                      "Add to group by or wrap in first() if you don't care which value you get.")
-                case e: Attribute => // OK
                 case e if groupingExprs.contains(e) => // OK
                 case e if e.references.isEmpty => // OK
                 case e => e.children.foreach(checkValidAggregateExpression)