[GitHub] spark pull request: [SPARK-4599] [Build] [SQL] add hive profile in...

2014-11-26 Thread pwendell
Github user pwendell commented on a diff in the pull request: https://github.com/apache/spark/pull/3450#discussion_r20980682 --- Diff: pom.xml --- @@ -1394,7 +1394,7 @@ /dependencies /profile profile - idhive-thriftserver/id +

[GitHub] spark pull request: [SPARK-4599] [Build] [SQL] add hive profile in...

2014-11-26 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/3450#issuecomment-64750619 We don't want to always build the thriftserver when hive support is enabled. Please see my e-mail to the dev list for more details. --- If your project is set up for

[GitHub] spark pull request: [SPARK-4599] [Build] [SQL] add hive profile in...

2014-11-26 Thread adrian-wang
Github user adrian-wang commented on the pull request: https://github.com/apache/spark/pull/3450#issuecomment-64753760 Thanks for the explanation, I'm closing it. --- 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-4599] [Build] [SQL] add hive profile in...

2014-11-26 Thread adrian-wang
Github user adrian-wang closed the pull request at: https://github.com/apache/spark/pull/3450 --- 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

[GitHub] spark pull request: [SPARK-4599] [Build] [SQL] add hive profile in...

2014-11-25 Thread adrian-wang
GitHub user adrian-wang opened a pull request: https://github.com/apache/spark/pull/3450 [SPARK-4599] [Build] [SQL] add hive profile in root pom This is what it was after #2685 , but seems reset by #3159 You can merge this pull request into a Git repository by running: $ git

[GitHub] spark pull request: [SPARK-4599] [Build] [SQL] add hive profile in...

2014-11-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3450#issuecomment-64327986 [Test build #23834 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/23834/consoleFull) for PR 3450 at commit

[GitHub] spark pull request: [SPARK-4599] [Build] [SQL] add hive profile in...

2014-11-25 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/3450#discussion_r20849854 --- Diff: pom.xml --- @@ -1394,7 +1394,7 @@ /dependencies /profile profile - idhive-thriftserver/id + idhive/id

[GitHub] spark pull request: [SPARK-4599] [Build] [SQL] add hive profile in...

2014-11-25 Thread adrian-wang
Github user adrian-wang commented on a diff in the pull request: https://github.com/apache/spark/pull/3450#discussion_r20850064 --- Diff: pom.xml --- @@ -1394,7 +1394,7 @@ /dependencies /profile profile - idhive-thriftserver/id +

[GitHub] spark pull request: [SPARK-4599] [Build] [SQL] add hive profile in...

2014-11-25 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3450#issuecomment-64376530 [Test build #23834 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/23834/consoleFull) for PR 3450 at commit

[GitHub] spark pull request: [SPARK-4599] [Build] [SQL] add hive profile in...

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