Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-20725][SQL][BRANCH-2.1] partial aggregate should behave correctly for sameResult #17975

Closed
wants to merge 1 commit into from

Conversation

cloud-fan
Copy link
Contributor

What changes were proposed in this pull request?

this backports #17964 to 2.1

@cloud-fan
Copy link
Contributor Author

cc @hvanhovell

@SparkQA
Copy link

SparkQA commented May 14, 2017

Test build #76904 has finished for PR 17975 at commit c4d1679.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@gatorsmile
Copy link
Member

We did not backport #17541 to 2.1. Is it still OK to backport this?

@cloud-fan
Copy link
Contributor Author

seems there is a bug if we backport this without #17541 , cc @hvanhovell shall we also backport #17541 ? Or leave branch-2.1 along as this is not a critical bug?

@cloud-fan cloud-fan closed this May 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants