Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SQL] [SPARK-8064] Fixes Parquet dependency issue for Hive 1.2.1 upgrade #7867

Closed
wants to merge 65 commits into from

Conversation

liancheng
Copy link
Contributor

This is a fork of PR #7191 authored by @steveloughran. This PR fixes the Parquet dependency issue, which causes exception while creating metastore Parquet tables due to missing Hive Parquet SerDe classes.

The missing classes originally come from com.twitter:parquet-hadoop-bundle:1.6.0, which should be a transitive dependency of org.apache.hive:hive-exec:1.2.1.spark, but somehow gets evicted in SBT build. This PR adds it back as a dependency of spark-hive.

Please refer to [this failed Jenkins build] for details of related test failures.

…ts pass instead of timing out: undetected regression?
…essages to say "w/Hive 1.2.1" in console output
…in the raised exception; this ensures that the stdout/stderr is included in jenkins reports, so it becomes possible to diagnose the cause.
…e rest of the work; can be backported to earlier versions
…ure that jersey server is on classpath on hadoop versions < 2.6
…till no obvious reason why jersey server code in spark-assembly isn't being picked up -it hasn't been shaded
steveloughran and others added 26 commits August 1, 2015 10:06
…nings. It's all autogenerated code, so still ugly.
-things that aren't needed are excluded
-sql/hive built with ivy is in sync with the maven reconciliation policy, rather than latest-first
…ed references in the hive- packages; this keeps mvn and ivy resolution compatible, as the reconciliation policy is "by hand"
… This one has the protobuf-shaded hive-exec jar
@SparkQA
Copy link

SparkQA commented Aug 2, 2015

Test build #39426 has finished for PR 7867 at commit dcbb391.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@liancheng
Copy link
Contributor Author

Closing this since it's merged into PR #7191.

@liancheng liancheng closed this Aug 3, 2015
@liancheng liancheng deleted the hive-1.2.1/parquet-deps-fix branch August 3, 2015 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants