Skip to content

Conversation

@sbcd90
Copy link
Contributor

@sbcd90 sbcd90 commented Apr 20, 2016

What changes were proposed in this pull request?

3 testcases namely,

"count is partially aggregated"
"count distinct is partially aggregated"
"mixed aggregates are partially aggregated"

were failing when running PlannerSuite individually.
The PR provides a fix for this.

How was this patch tested?

unit tests

(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

@rxin
Copy link
Contributor

rxin commented Apr 20, 2016

cc @marmbrus

@marmbrus
Copy link
Contributor

OK to test

@sbcd90
Copy link
Contributor Author

sbcd90 commented Apr 20, 2016

Jenkins test this please

@SparkQA
Copy link

SparkQA commented Apr 20, 2016

Test build #2837 has finished for PR 12532 at commit 5f5238e.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@sbcd90
Copy link
Contributor Author

sbcd90 commented Apr 20, 2016

Hi @rxin ,

Can this be merged?

@rxin
Copy link
Contributor

rxin commented Apr 20, 2016

Merging in master. Thanks.

@asfgit asfgit closed this in fd82681 Apr 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants