Skip to content

Conversation

@gvramana
Copy link
Contributor

@gvramana gvramana commented Apr 7, 2015

TranslatingClassLoader is used for Spark-shell, while current hive's add jar can work only with URLClassLoader.
So jar has to be directly added to spark driver's class loader or its parent loader, in case of spark-shell

gvramana added 2 commits April 7, 2015 20:07
These tests will run only when compiled with -Phive
@gvramana
Copy link
Contributor Author

Hi, Can some one verify this patch. Thanks

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can't make this change as I think it means that we will always build hive with the spark assembly, correct?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, dependency is added in hive profile only. So if assembly is built with -Phive option then hive dependency is added to repl, so that hive is available in class path for repl.
If assembly is built without -Phive option then dependency is not added and the testcase is ignored. Testcase also checks runtime if hiveContext class is available, if not testcase is ignored.
I have manually tested both the cases of building with -Phive and without hive. It will not impact assembly creation.

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@marmbrus
Copy link
Contributor

marmbrus commented Sep 3, 2015

Thanks for working on this. Can we close this issue until you have time to bring it up to date?

@asfgit asfgit closed this in 804a012 Sep 4, 2015
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.

3 participants