Skip to content

Conversation

@hvanhovell
Copy link
Contributor

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 #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%.

@dongjoon-hyun
Copy link
Member

+1 @hvanhovell .
Since Jenkins does not run scala-2.10 compilation, I ran the build locally on Ubuntu/JDK7/Scala 2.10 (just for double-check for this PR.)

@SparkQA
Copy link

SparkQA commented Jun 22, 2016

Test build #61059 has finished for PR 13854 at commit db3161b.

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

@hvanhovell
Copy link
Contributor Author

Merging to master.

@asfgit asfgit closed this in 4f869f8 Jun 22, 2016
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.

4 participants