GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/11427
[SPARK-13549] [SQL] Refactor the Optimizer Rule CollapseProject
#### What changes were proposed in this pull request?
The PR https://github.com/apache/spark/pull/10541 changed the rule
`CollapseProject` by enabling collapsing `Project` into `Aggregate`. It leaves
a to-do item to remove the duplicate code. This PR is to finish this to-do
item. Also added a test case for covering this change.
#### How was this patch tested?
Added a new test case.
@liancheng Could you check if the code refactoring is fine? Thanks!
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/gatorsmile/spark collapseProjectRefactor
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/11427.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #11427
----
commit 9989a58da44b2b97a660cf03a330408a46f5036a
Author: gatorsmile <gatorsm...@gmail.com>
Date: 2016-02-29T06:29:45Z
refactoring CollapseProject
commit d132c5825b103644bfd9c9b41d91424e38a63940
Author: gatorsmile <gatorsm...@gmail.com>
Date: 2016-02-29T07:19:01Z
added a test case.
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
