GitHub user hvanhovell opened a pull request:

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

    [SPARK-15956] [SQL] When unwrapping ORC avoid pattern matching at runtime

    ## What changes were proposed in this pull request?
    
    Extend the returning of unwrapper functions from primitive types to all 
types.
    
    This PR is based on https://github.com/apache/spark/pull/13676. It only 
fixes a bug with scala-2.10 compilation. All credit should go to @dafrista.
    
    ## How was this patch tested?
    
    The patch should pass all unit tests. Reading ORC files with non-primitive 
types with this change reduced the read time by ~15%.

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

    $ git pull https://github.com/hvanhovell/spark SPARK-15956-scala210

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

    https://github.com/apache/spark/pull/13854.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 #13854
    
----
commit bdd188546c6043b694e2b1e023a4e78f1e5fffdb
Author: Brian Cho <[email protected]>
Date:   2016-06-13T20:36:17Z

    Avoid pattern matching in runtime when unwrapping ORC.

commit ca44f12a04312ced5f5e861df7dbe85d56f8b4af
Author: Brian Cho <[email protected]>
Date:   2016-06-15T05:28:18Z

    Address comments: remove unwrap method.

commit 28a02f3afd362fe941e583cbc5ccbc0a9aefb4f2
Author: Brian Cho <[email protected]>
Date:   2016-06-20T07:18:26Z

    Address comments: evaluate constant outside function, clean-up

----


---
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 [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to