Skip to content

Conversation

@rxin
Copy link
Contributor

@rxin rxin commented Aug 20, 2015

This improves performance by ~ 20 - 30% in one of my local test and should fix the performance regression from 1.4 to 1.5 on ss_max.

…g key in aggregation.

This improves performance by ~ 20% in one of my local test.
@SparkQA
Copy link

SparkQA commented Aug 20, 2015

Test build #41309 has finished for PR 8332 at commit 4dc723d.

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

@yhuai
Copy link
Contributor

yhuai commented Aug 20, 2015

LGTM. merging to master and branch 1.5.

asfgit pushed a commit that referenced this pull request Aug 20, 2015
…ng key in aggregation.

This improves performance by ~ 20 - 30% in one of my local test and should fix the performance regression from 1.4 to 1.5 on ss_max.

Author: Reynold Xin <[email protected]>

Closes #8332 from rxin/SPARK-10100.

(cherry picked from commit b4f4e91)
Signed-off-by: Yin Huai <[email protected]>
@asfgit asfgit closed this in b4f4e91 Aug 20, 2015
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