GitHub user davies opened a pull request:

    https://github.com/apache/spark/pull/10977

    [SPARK-13031] [SQL] cleanup codegen and improve test coverage

    1. enable whole stage codegen during tests even there is only one operator 
supports that.
    2. split doProduce() into two APIs: upstream() and doProduce()
    3. generate prefix for fresh names of each operator
    4. pass UnsafeRow to parent directly (avoid getters and create UnsafeRow 
again)
    5. fix bugs and tests.
    
    This PR re-open #10944 and fix the bug.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/davies/spark gen_refactor

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/10977.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 #10977
    
----
commit b4db00675bc3c51ddf8735cace522a5d771cf7e2
Author: Davies Liu <dav...@databricks.com>
Date:   2016-01-27T07:43:40Z

    cleanup whole stage codegen

commit 70a7c7edd1988c7dd69bccc8e563c9943775bd2c
Author: Davies Liu <dav...@databricks.com>
Date:   2016-01-27T23:22:33Z

    improve stddev and variance

commit 951e2cd8de2c6b2f5b5bd1f5dbdb6b6fad6bc4a4
Author: Davies Liu <dav...@databricks.com>
Date:   2016-01-29T06:24:05Z

    fix aggregation without functions

----


---
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

Reply via email to