Skip to content

Conversation

cloud-fan
Copy link
Contributor

What changes were proposed in this pull request?

according to the discussion in the original PR #10896 and the new approach PR #14876 , we decided to revert these 2 PRs and go with the new approach.

How was this patch tested?

N/A

… data already clustered with group-by keys"

This reverts commit 2b0cc4e.
@cloud-fan
Copy link
Contributor Author

cc @maropu @hvanhovell @yhuai

@maropu after this, do you wanna take over #14876 and keep working on it? thanks!

@maropu
Copy link
Member

maropu commented Sep 1, 2016

@cloud-fan yea, thanks! If no problem, I'll take it over.

@SparkQA
Copy link

SparkQA commented Sep 1, 2016

Test build #64752 has finished for PR 14909 at commit 78cf93b.

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

@cloud-fan
Copy link
Contributor Author

merging to master!

@asfgit asfgit closed this in aaf632b Sep 1, 2016
@maropu
Copy link
Member

maropu commented Sep 1, 2016

okay, thanks!

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.

3 participants