[GitHub] spark pull request: SPARK-5143 [BUILD] [WIP] spark-network-yarn 2....

2015-03-04 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/4876#issuecomment-77246969 I commented on the JIRA. This LGTM as an immediate fix - clearly the property is not correct in the published 2.11 poms. --- If your project is set up for it, you can

[GitHub] spark pull request: SPARK-5143 [BUILD] [WIP] spark-network-yarn 2....

2015-03-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/4876 --- 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-5143 [BUILD] [WIP] spark-network-yarn 2....

2015-03-04 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/4876#issuecomment-77120315 Don't worry about `-e`, I think. Does the command work to replace just the first occurrence on GNU `sed`? I can dig out a linux box but just wondered if you knew already.

[GitHub] spark pull request: SPARK-5143 [BUILD] [WIP] spark-network-yarn 2....

2015-03-04 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/4876#discussion_r25760252 --- Diff: dev/change-version-to-2.10.sh --- @@ -17,4 +17,7 @@ # limitations under the License. # find . -name 'pom.xml' | grep -v target \ -

[GitHub] spark pull request: SPARK-5143 [BUILD] [WIP] spark-network-yarn 2....

2015-03-04 Thread ScrapCodes
Github user ScrapCodes commented on the pull request: https://github.com/apache/spark/pull/4876#issuecomment-77168087 Hmm.. I was trusting our effective pom trick was working properly. Forgot the JIRA id for that though. On Mar 4, 2015 6:10 PM, Sean Owen

[GitHub] spark pull request: SPARK-5143 [BUILD] [WIP] spark-network-yarn 2....

2015-03-04 Thread ScrapCodes
Github user ScrapCodes commented on the pull request: https://github.com/apache/spark/pull/4876#issuecomment-77169399 Based on your comments and this issue I am going to take a deeper look tomorrow again. Meanwhile if you are rushed with respect to this PR. Looks like a good

[GitHub] spark pull request: SPARK-5143 [BUILD] [WIP] spark-network-yarn 2....

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

[GitHub] spark pull request: SPARK-5143 [BUILD] [WIP] spark-network-yarn 2....

2015-03-03 Thread srowen
GitHub user srowen opened a pull request: https://github.com/apache/spark/pull/4876 SPARK-5143 [BUILD] [WIP] spark-network-yarn 2.11 depends on spark-network-shuffle 2.10 Update `scala.binary.version` prop in POM when switching between Scala 2.10/2.11 @ScrapCodes for

[GitHub] spark pull request: SPARK-5143 [BUILD] [WIP] spark-network-yarn 2....

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

[GitHub] spark pull request: SPARK-5143 [BUILD] [WIP] spark-network-yarn 2....

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

[GitHub] spark pull request: SPARK-5143 [BUILD] [WIP] spark-network-yarn 2....

2015-03-03 Thread ScrapCodes
Github user ScrapCodes commented on a diff in the pull request: https://github.com/apache/spark/pull/4876#discussion_r25752030 --- Diff: dev/change-version-to-2.10.sh --- @@ -17,4 +17,7 @@ # limitations under the License. # find . -name 'pom.xml' | grep -v target \

[GitHub] spark pull request: SPARK-5143 [BUILD] [WIP] spark-network-yarn 2....

2015-03-03 Thread ScrapCodes
Github user ScrapCodes commented on the pull request: https://github.com/apache/spark/pull/4876#issuecomment-77097244 I am not a sed wizard, was just going through the man page. It does not say anything about -e being POSIX or non POSIX - sadly!. I do not have a mac, so no clues