Skip to content

Conversation

@witgo
Copy link
Contributor

@witgo witgo commented May 26, 2014

No description provided.

@AmplabJenkins
Copy link

Merged build triggered.

@AmplabJenkins
Copy link

Merged build started.

@AmplabJenkins
Copy link

Merged build finished. All automated tests passed.

@AmplabJenkins
Copy link

All automated tests passed.
Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/15207/

@rxin
Copy link
Contributor

rxin commented May 26, 2014

Thanks. I've merged this into master.

@asfgit asfgit closed this in bee6c4f May 26, 2014
@witgo witgo deleted the scalastyle branch May 27, 2014 15:50
pdeyhim pushed a commit to pdeyhim/spark-1 that referenced this pull request Jun 25, 2014
Author: witgo <[email protected]>

Closes apache#884 from witgo/scalastyle and squashes the following commits:

4b08ae4 [witgo] Fix scalastyle warnings in yarn alpha
xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
Author: witgo <[email protected]>

Closes apache#884 from witgo/scalastyle and squashes the following commits:

4b08ae4 [witgo] Fix scalastyle warnings in yarn alpha
wangyum added a commit that referenced this pull request May 26, 2023
…tributes (#884)

* Remove the aggregation from left semi/anti join if the same aggregation has already been done on left side

* Add more test

* grouping -> groupingExps

* Add DistinctAttributesVisitor

* Fix test name

* Improve DistinctAttributesVisitor

* Fix test.

* DistinctKeyVisitor

* Address comments

* Fix scala 2.13

* Address comments

* Address all comments

* Address all comments

* fix

* fix test

* [SPARK-38489][SQL] Aggregate.groupOnly support foldable expressions

### What changes were proposed in this pull request?

This pr makes `Aggregate.groupOnly` support foldable expressions.

### Why are the changes needed?

Improve query performance.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Unit test.

Closes #35795 from wangyum/SPARK-38489.

Authored-by: Yuming Wang <[email protected]>
Signed-off-by: Wenchen Fan <[email protected]>

(cherry picked from commit bcf7849)

* Remove

* fix

* Update DistinctKeyVisitor.scala
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