Skip to content

Commit

Permalink
fix for grouping__id
Browse files Browse the repository at this point in the history
  • Loading branch information
scwf committed Feb 12, 2015
1 parent f279a71 commit fa38881
Showing 1 changed file with 3 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ class Analyzer(catalog: Catalog,
// of the result of cascading resolution failures.
plan.foreachUp {
case operator: LogicalPlan =>
operator transformAllExpressions {
operator transformExpressionsUp {
case a: Attribute if !a.resolved =>
val from = operator.inputSet.map(_.name).mkString(", ")
failAnalysis(s"cannot resolve '${a.prettyString}' given input columns $from")
Expand Down Expand Up @@ -125,7 +125,8 @@ class Analyzer(catalog: Catalog,

cleaned.foreach(checkValidAggregateExpression)

case o if o.children.nonEmpty && !o.references.subsetOf(o.inputSet) =>
case o if o.children.nonEmpty &&
!o.references.filter(_.name != "grouping__id").subsetOf(o.inputSet) =>
val missingAttributes = (o.references -- o.inputSet).map(_.prettyString).mkString(",")
val input = o.inputSet.map(_.prettyString).mkString(",")

Expand Down

0 comments on commit fa38881

Please sign in to comment.