[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-06-04 Thread dougb
Github user dougb commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-108910838 FYI, this change broke getting a DF via `sqlContext.table(database.table)` which worked from `1.3.0` up until this pr. --- If your project is set up for it, you

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-06-01 Thread jeanlyn
Github user jeanlyn commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-107415482 I set the metastore to `0.12.0` by follow steps ,but get classnotdef exception: * I chang the `spark.sql.hive.metastore.version` in `spark-defaults.conf` to

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-06-01 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-107606325 @jeanlyn The warning message related `get_functions` is expected if you are using a Hive 0.13.1 metastore client to connect to a Hive 0.12.0 metastore server because

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-06-01 Thread jeanlyn
Github user jeanlyn commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-107459171 I can use the build in class(0.13.1) to connect `0.12.0` metastore correctly except some warn and error which does not effect running ``` 5/06/01 21:20:09 WARN

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-06-01 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-107733461 @jeanlyn I think you are hitting https://issues.apache.org/jira/browse/SPARK-8020. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-21 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-104504904 Yes, please file a JIRA. On May 21, 2015 9:29 PM, coderfi notificati...@github.com wrote: Has this been tested with mapr4? When running under

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-21 Thread coderfi
Github user coderfi commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-104503508 Has this been tested with mapr4? When running under python, I am running into an exception Caused by: java.lang.UnsatisfiedLinkError: Native Library

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-21 Thread coderfi
Github user coderfi commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-104511653 K. Filed as https://issues.apache.org/jira/browse/SPARK-7819 thank you for looking into this! Fi --- If your project is set up for it, you can reply

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-99973577 Seems we can remove the lock at https://github.com/marmbrus/spark/blob/useIsolatedClient/sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala#L73.

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29883765 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala --- @@ -144,39 +147,43 @@ private[hive] class HiveMetastoreCatalog(hive:

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29883543 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -93,9 +99,129 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29885759 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala --- @@ -1080,6 +1149,15 @@

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100026910 I left a few comments mostly around the configuration parts of the change. I'm not really familiar with the rest to comment. --- If your project is set up for it, you

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29900100 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -93,9 +99,129 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29901320 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -93,9 +99,129 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100041690 Thanks for the comments @vanzin, I think I have addressed everything except the ability to configure hive through `spark.hadoop.*`. --- If your project is set up for

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100041602 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29906395 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -93,9 +100,113 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29907530 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -93,9 +100,113 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100052138 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100052136 [Test build #32165 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32165/consoleFull) for PR 5876 at commit

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100052901 [Test build #32167 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32167/consoleFull) for PR 5876 at commit

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100047039 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100047037 Merged build finished. Test FAILed. --- 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-6908] [SQL] Use isolated Hive client

2015-05-07 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29907415 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -93,9 +100,113 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100051873 [Test build #32165 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32165/consoleFull) for PR 5876 at commit

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100052386 test this please --- 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-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29906552 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -93,9 +100,113 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100052137 Merged build finished. Test FAILed. --- 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-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100047386 Just a couple of comments left, otherwise the parts I commented on look good. Thanks! --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29907032 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -93,9 +100,113 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29907285 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -93,9 +100,113 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100051672 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100051683 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100052711 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100052724 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29895935 --- Diff: sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/SparkSQLCLIDriver.scala --- @@ -91,10 +100,14 @@ private[hive] object

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29896662 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -93,9 +99,129 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29896733 --- Diff: sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/SparkSQLEnv.scala --- @@ -51,6 +52,12 @@ private[hive] object

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29897294 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -93,9 +99,129 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100042338 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-6908] [SQL] Use isolated Hive client

2015-05-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100042457 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100077600 [Test build #32167 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32167/consoleFull) for PR 5876 at commit

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

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

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100077609 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-6908] [SQL] Use isolated Hive client

2015-05-07 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100077855 LGTM. I will merging it to master and branch 1.4. --- 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-6908] [SQL] Use isolated Hive client

2015-05-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/5876 --- 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-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29895833 --- Diff: sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/SparkSQLCLIDriver.scala --- @@ -74,7 +74,16 @@ private[hive] object

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29895762 --- Diff: sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/SparkSQLCLIDriver.scala --- @@ -74,7 +74,16 @@ private[hive] object

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29896863 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -93,9 +99,129 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29897419 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -93,9 +99,129 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29898475 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/test/TestHive.scala --- @@ -70,24 +71,22 @@ class TestHiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29899653 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -93,9 +99,129 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100041616 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100042324 [Test build #32149 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32149/consoleFull) for PR 5876 at commit

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

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

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29896544 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -93,9 +99,129 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29896591 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -93,9 +99,129 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29900315 --- Diff: sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/SparkSQLCLIDriver.scala --- @@ -74,7 +74,16 @@ private[hive] object

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100042454 Merged build finished. Test FAILed. --- 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-6908] [SQL] Use isolated Hive client

2015-05-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100042449 [Test build #32157 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32157/consoleFull) for PR 5876 at commit

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29905368 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -439,7 +455,21 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100045495 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100045487 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29896241 --- Diff: sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/SparkSQLEnv.scala --- @@ -51,6 +52,12 @@ private[hive] object

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29897174 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -93,9 +99,129 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29898324 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/CreateTableAsSelect.scala --- @@ -39,17 +39,34 @@ import

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100041843 [Test build #32157 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32157/consoleFull) for PR 5876 at commit

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29905416 --- Diff: sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/SparkSQLCLIDriver.scala --- @@ -74,7 +74,17 @@ private[hive] object

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29905330 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -439,7 +455,21 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100010327 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29899815 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -93,9 +99,129 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29899875 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -93,9 +99,129 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29903763 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -93,9 +99,129 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29905464 --- Diff: sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/SparkSQLCLIDriver.scala --- @@ -74,7 +74,17 @@ private[hive] object

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100010571 [Test build #32149 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32149/consoleFull) for PR 5876 at commit

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-100010299 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29900905 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -93,9 +99,129 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-07 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/5876#discussion_r29871007 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveContext.scala --- @@ -93,9 +99,129 @@ class HiveContext(sc: SparkContext) extends

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-99347842 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-99347799 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-99348760 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-99348758 Merged build finished. Test FAILed. --- 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-6908] [SQL] Use isolated Hive client

2015-05-06 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-99348747 [Test build #31958 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31958/consoleFull) for PR 5876 at commit

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-99354002 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-06 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-99353991 [Test build #31961 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31961/consoleFull) for PR 5876 at commit

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-06 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-99348051 [Test build #31958 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31958/consoleFull) for PR 5876 at commit

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-99353998 Merged build finished. Test FAILed. --- 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-6908] [SQL] Use isolated Hive client

2015-05-06 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-99353434 [Test build #31961 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31961/consoleFull) for PR 5876 at commit

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-99353383 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-99353401 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-99642842 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-99642855 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-99633112 Build finished. Test FAILed. --- 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

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-99634906 Merged build triggered. --- 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

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-99634922 Merged build started. --- 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

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-99640561 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-06 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-99640498 [Test build #32030 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32030/consoleFull) for PR 5876 at commit

[GitHub] spark pull request: [SPARK-6908] [SQL] Use isolated Hive client

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-99640557 Merged build finished. Test FAILed. --- 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-6908] [SQL] Use isolated Hive client

2015-05-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5876#issuecomment-99643323 Merged build finished. Test FAILed. --- 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

  1   2   >