[SPARK-6505][SQL]Remove the reflection call in HiveFunctionWrapper

[baishuo]

according @liancheng‘s comment in https://issues.apache.org/jira/browse/SPARK-6505, this patch remove the reflection call in HiveFunctionWrapper, and implement the functions named "deserializeObjectByKryo" and "serializeObjectByKryo" according the functions with the save name in
org.apache.hadoop.hive.ql.exec.Utilities.java

[AmplabJenkins]

Can one of the admins verify this patch?

[liancheng]

ok to test

[SparkQA]

Test build #30848 has started for PR 5660 at commit 0b522a7.

[liancheng]

Would like to add some background about this change for future reference:

Spark SQL repackaged Hive jars to remove unnecessary shading and dependencies. One of them is Kryo. Hive shades Kryo to package org.apache.hive.com.esotericsoftware. Normally this should be fine. However, MapR replaces Spark SQL's repackaged Hive dependencies with genuine Hive packages, thus the reflection call mentioned in this PR couldn't find Kryo because it's moved into another package. On the other hand, the two reflected methods are actually quite simple. That's why we chose to reimplement them in Spark SQL instead of using them via reflection.

[liancheng]

Move imports to the header of the file.

[liancheng]

add to whitelist

[SparkQA]

Test build #30849 has started for PR 5660 at commit 0b522a7.

[SparkQA]

Test build #30848 has finished for PR 5660 at commit 0b522a7.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.
• This patch does not change any dependencies.

[AmplabJenkins]

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30848/
Test PASSed.

[SparkQA]

Test build #30849 has finished for PR 5660 at commit 0b522a7.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.
• This patch does not change any dependencies.

[AmplabJenkins]

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30849/
Test PASSed.

[SparkQA]

Test build #30892 has started for PR 5660 at commit ae61ec4.

[SparkQA]

Test build #30892 has finished for PR 5660 at commit ae61ec4.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.
• This patch does not change any dependencies.

[AmplabJenkins]

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30892/
Test PASSed.

[baishuo]

can this patch be merged? :)

[liancheng]

Thanks for working on this! I'm merging this to master.

[srowen]

@baishuo what's your Apache JIRA username? I can assign you to the JIRA to credit you.

[baishuo]

hi @srowen , my jira name is also baishuo
https://issues.apache.org/jira/secure/ViewProfile.jspa?name=baishuo
thanks :)