Skip to content

Commit

Permalink
Fix tests now that the planner is better
Browse files Browse the repository at this point in the history
  • Loading branch information
marmbrus committed Aug 18, 2014
1 parent 41fbd1d commit d494598
Showing 1 changed file with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -45,16 +45,16 @@ class PlannerSuite extends FunSuite {
assert(aggregations.size === 2)
}

test("count distinct is not partially aggregated") {
test("count distinct is partially aggregated") {
val query = testData.groupBy('value)(CountDistinct('key :: Nil)).queryExecution.analyzed
val planned = HashAggregation(query)
assert(planned.isEmpty)
assert(planned.nonEmpty)
}

test("mixed aggregates are not partially aggregated") {
test("mixed aggregates are partially aggregated") {
val query =
testData.groupBy('value)(Count('value), CountDistinct('key :: Nil)).queryExecution.analyzed
val planned = HashAggregation(query)
assert(planned.isEmpty)
assert(planned.nonEmpty)
}
}

0 comments on commit d494598

Please sign in to comment.