[GitHub] spark pull request: [SPARK-13648] Add Hive Cli to classes for isol...

2016-03-08 Thread preecet
Github user preecet commented on the pull request: https://github.com/apache/spark/pull/11495#issuecomment-193676822 Because ClientWrapper calls isInstanceOf[CliSessionState] and so needs to load the class CliSessionstate. In the testcase ClientWraper has been loaded by the isolated

[GitHub] spark pull request: [SPARK-13648] Add Hive Cli to classes for isol...

2016-03-08 Thread adrian-wang
Github user adrian-wang commented on the pull request: https://github.com/apache/spark/pull/11495#issuecomment-193670192 Why we need `hive-cli` here? This is not version specific. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark pull request: [SPARK-13648] Add Hive Cli to classes for isol...

2016-03-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/11495 --- 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

[GitHub] spark pull request: [SPARK-13648] Add Hive Cli to classes for isol...

2016-03-07 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/11495#issuecomment-193501590 Thanks, merging to master and 1.6. --- 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

[GitHub] spark pull request: [SPARK-13648] Add Hive Cli to classes for isol...

2016-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11495#issuecomment-192049695 Merged build finished. Test PASSed. --- 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

[GitHub] spark pull request: [SPARK-13648] Add Hive Cli to classes for isol...

2016-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11495#issuecomment-192049696 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13648] Add Hive Cli to classes for isol...

2016-03-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11495#issuecomment-192049471 **[Test build #52418 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52418/consoleFull)** for PR 11495 at commit

[GitHub] spark pull request: [SPARK-13648] Add Hive Cli to classes for isol...

2016-03-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11495#issuecomment-192023050 **[Test build #52418 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52418/consoleFull)** for PR 11495 at commit

[GitHub] spark pull request: [SPARK-13648] Add Hive Cli to classes for isol...

2016-03-03 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/11495#issuecomment-192022855 LGTM pending tests --- 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

[GitHub] spark pull request: [SPARK-13648] Add Hive Cli to classes for isol...

2016-03-03 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/11495#issuecomment-192022772 ok to test --- 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

[GitHub] spark pull request: [SPARK-13648] Add Hive Cli to classes for isol...

2016-03-03 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/11495#issuecomment-192012123 cc @marmbrus --- 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

[GitHub] spark pull request: [SPARK-13648] Add Hive Cli to classes for isol...

2016-03-03 Thread preecet
Github user preecet commented on the pull request: https://github.com/apache/spark/pull/11495#issuecomment-191797212 Copy of recent developemrnt mailing list comment. I have been testing 1.6.1RC1 using the IBM Java SDK. I notice a problem ( with the

[GitHub] spark pull request: [SPARK-13648] Add Hive Cli to classes for isol...

2016-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11495#issuecomment-191749270 Can one of the admins verify this patch? --- 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

[GitHub] spark pull request: [SPARK-13648] Add Hive Cli to classes for isol...

2016-03-03 Thread preecet
GitHub user preecet opened a pull request: https://github.com/apache/spark/pull/11495 [SPARK-13648] Add Hive Cli to classes for isolated classloader ## What changes were proposed in this pull request? Adding the hive-cli classes to the classloader ## How was