[GitHub] spark pull request: [SPARK-7042] [BUILD] use the standard akka art...

2015-06-11 Thread nchammas
Github user nchammas commented on the pull request: https://github.com/apache/spark/pull/6492#issuecomment-111295751 OK, I confirmed that using Maven 3.3.3 resolves my issue. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] spark pull request: [SPARK-7042] [BUILD] use the standard akka art...

2015-06-11 Thread nchammas
Github user nchammas commented on the pull request: https://github.com/apache/spark/pull/6492#issuecomment-57597 @kostya-sh @zsxwing - I was using [Maven 3.2.5](https://github.com/databricks/spark-perf/blob/8a966c48ec84a60bd8b139e684d8da63c8d43e0f/bin/spark-ec2/install-maven#L5).

[GitHub] spark pull request: [SPARK-7042] [BUILD] use the standard akka art...

2015-06-11 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/6492#issuecomment-111030683 So let's require Maven 3.3 -- problem solved! I'm going to just say it: what about dropping Hadoop 1.x support in Spark 1.5? Java 6 support is going away too. -

[GitHub] spark pull request: [SPARK-7042] [BUILD] use the standard akka art...

2015-06-10 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/6492#issuecomment-111013435 3.2.5 has the same issue, too ``` mvn -Phadoop-1 dependency:tree | grep akka [INFO] +- org.spark-project.akka:akka-remote_2.10:jar:2.3.4-spark:compile [INF

[GitHub] spark pull request: [SPARK-7042] [BUILD] use the standard akka art...

2015-06-10 Thread kostya-sh
Github user kostya-sh commented on the pull request: https://github.com/apache/spark/pull/6492#issuecomment-111001101 @nchammas, also what version of maven did you use to build Spark? A quick test shows that at least maven-3.1.1 is broken (by not taking active profile into acc

[GitHub] spark pull request: [SPARK-7042] [BUILD] use the standard akka art...

2015-06-10 Thread kostya-sh
Github user kostya-sh commented on the pull request: https://github.com/apache/spark/pull/6492#issuecomment-110994822 @nchammas, what errors are you getting? --- 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-7042] [BUILD] use the standard akka art...

2015-06-10 Thread nchammas
Github user nchammas commented on the pull request: https://github.com/apache/spark/pull/6492#issuecomment-110949299 Heads up: This PR appears to have broken Spark when built against Hadoop 1. I've been having trouble coming up with a simple repro, but [spark-perf](https://github.com/

[GitHub] spark pull request: [SPARK-7042] [BUILD] use the standard akka art...

2015-06-08 Thread kostya-sh
Github user kostya-sh closed the pull request at: https://github.com/apache/spark/pull/6492 --- 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 e

[GitHub] spark pull request: [SPARK-7042] [BUILD] use the standard akka art...

2015-06-08 Thread kostya-sh
Github user kostya-sh commented on the pull request: https://github.com/apache/spark/pull/6492#issuecomment-109978478 @srowen, thank you. Closing the pull request now. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If yo

[GitHub] spark pull request: [SPARK-7042] [BUILD] use the standard akka art...

2015-06-08 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/6492#issuecomment-109906905 @kostya-sh I merged this, but it didn't auto-close due to a sync problem. You can close this PR. --- If your project is set up for it, you can reply to this email and ha

[GitHub] spark pull request: [SPARK-7042] [BUILD] use the standard akka art...

2015-06-06 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/6492#issuecomment-109606433 Yep LGTM --- 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 enab

[GitHub] spark pull request: [SPARK-7042] [BUILD] use the standard akka art...

2015-06-06 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/6492#issuecomment-109553915 I'm inclined to pull this in if in fact it works with Scala 2.11 this time around, as reported on the JIRA. 2.3.11 is claimed to be a drop in replacement for earlier 2.3.

[GitHub] spark pull request: [SPARK-7042] [BUILD] use the standard akka art...

2015-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6492#issuecomment-106763128 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 project

[GitHub] spark pull request: [SPARK-7042] [BUILD] use the standard akka art...

2015-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6492#issuecomment-106763117 [Test build #33728 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33728/consoleFull) for PR 6492 at commit [`dc195b0`](https://git

[GitHub] spark pull request: [SPARK-7042] [BUILD] use the standard akka art...

2015-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6492#issuecomment-106741303 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-7042] [BUILD] use the standard akka art...

2015-05-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6492#issuecomment-106741360 [Test build #33728 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33728/consoleFull) for PR 6492 at commit [`dc195b0`](https://gith

[GitHub] spark pull request: [SPARK-7042] [BUILD] use the standard akka art...

2015-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6492#issuecomment-106741287 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 h

[GitHub] spark pull request: [SPARK-7042] [BUILD] use the standard akka art...

2015-05-29 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/6492#issuecomment-106739279 Jenkins, add to whitelist --- 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 th

[GitHub] spark pull request: [SPARK-7042] [BUILD] use the standard akka art...

2015-05-29 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/6492#issuecomment-106739270 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 enab

[GitHub] spark pull request: [SPARK-7042] [BUILD] use the standard akka art...

2015-05-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6492#issuecomment-106723393 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 pr

[GitHub] spark pull request: [SPARK-7042] [BUILD] use the standard akka art...

2015-05-29 Thread kostya-sh
GitHub user kostya-sh opened a pull request: https://github.com/apache/spark/pull/6492 [SPARK-7042] [BUILD] use the standard akka artifacts with hadoop-2.x Both akka 2.3.x and hadoop-2.x use protobuf 2.5 so only hadoop-1 build needs custom 2.3.4-spark akka version that shades pro