[GitHub] spark pull request: [SPARK-6627] Some clean-up in shuffle code.

2015-04-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/5286 --- 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-6627] Some clean-up in shuffle code.

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

[GitHub] spark pull request: [SPARK-6627] Some clean-up in shuffle code.

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

[GitHub] spark pull request: [SPARK-6627] Some clean-up in shuffle code.

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

[GitHub] spark pull request: [SPARK-6627] Some clean-up in shuffle code.

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

[GitHub] spark pull request: [SPARK-6627] Some clean-up in shuffle code.

2015-04-01 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/5286#issuecomment-88663368 Jenkins, retest 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

[GitHub] spark pull request: [SPARK-6627] Some clean-up in shuffle code.

2015-04-01 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/5286#issuecomment-88728572 Jenkins, retest 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

[GitHub] spark pull request: [SPARK-6627] Some clean-up in shuffle code.

2015-04-01 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/5286#issuecomment-88728689 /cc @brennonyork - this is giving me strange dependency messages even though the patch doesn't touch any pom files. --- If your project is set up for it, you can reply

[GitHub] spark pull request: [SPARK-6627] Some clean-up in shuffle code.

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

[GitHub] spark pull request: [SPARK-6627] Some clean-up in shuffle code.

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

[GitHub] spark pull request: [SPARK-6627] Some clean-up in shuffle code.

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

[GitHub] spark pull request: [SPARK-6627] Some clean-up in shuffle code.

2015-03-31 Thread aarondav
Github user aarondav commented on the pull request: https://github.com/apache/spark/pull/5286#issuecomment-87972548 LGTM too --- 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-6627] Some clean-up in shuffle code.

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

[GitHub] spark pull request: [SPARK-6627] Some clean-up in shuffle code.

2015-03-31 Thread aarondav
Github user aarondav commented on a diff in the pull request: https://github.com/apache/spark/pull/5286#discussion_r27457202 --- Diff: core/src/main/scala/org/apache/spark/shuffle/IndexShuffleBlockManager.scala --- @@ -39,25 +41,18 @@ import org.apache.spark.storage._ //

[GitHub] spark pull request: [SPARK-6627] Some clean-up in shuffle code.

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

[GitHub] spark pull request: [SPARK-6627] Some clean-up in shuffle code.

2015-03-31 Thread aarondav
Github user aarondav commented on a diff in the pull request: https://github.com/apache/spark/pull/5286#discussion_r27457157 --- Diff: core/src/main/scala/org/apache/spark/shuffle/IndexShuffleBlockManager.scala --- @@ -123,3 +114,10 @@ class IndexShuffleBlockManager(conf:

[GitHub] spark pull request: [SPARK-6627] Some clean-up in shuffle code.

2015-03-31 Thread pwendell
Github user pwendell commented on a diff in the pull request: https://github.com/apache/spark/pull/5286#discussion_r27496946 --- Diff: core/src/main/scala/org/apache/spark/shuffle/IndexShuffleBlockManager.scala --- @@ -39,25 +41,18 @@ import org.apache.spark.storage._ //

[GitHub] spark pull request: [SPARK-6627] Some clean-up in shuffle code.

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

[GitHub] spark pull request: [SPARK-6627] Some clean-up in shuffle code.

2015-03-31 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/5286#issuecomment-88186520 Jenkins, retest this pleas.e --- 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-6627] Some clean-up in shuffle code.

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

[GitHub] spark pull request: [SPARK-6627] Some clean-up in shuffle code.

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