[GitHub] spark pull request: [SPARK-3970] Remove duplicate removal of local...

2014-10-26 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/2826#issuecomment-60514830 Is this patch ok to merge? --- 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-3970] Remove duplicate removal of local...

2014-10-26 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/2826#issuecomment-60527950 retest this please, just because it's been a while since jenkins last ran. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-3970] Remove duplicate removal of local...

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

[GitHub] spark pull request: [SPARK-3970] Remove duplicate removal of local...

2014-10-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/2826#issuecomment-60532427 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-3970] Remove duplicate removal of local...

2014-10-26 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/2826#issuecomment-60532421 **[Test build #22249 timed out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/22249/consoleFull)** for PR 2826 at commit

[GitHub] spark pull request: [SPARK-3970] Remove duplicate removal of local...

2014-10-26 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/2826#issuecomment-60532545 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 have this

[GitHub] spark pull request: [SPARK-3970] Remove duplicate removal of local...

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

[GitHub] spark pull request: [SPARK-3970] Remove duplicate removal of local...

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

[GitHub] spark pull request: [SPARK-3970] Remove duplicate removal of local...

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

[GitHub] spark pull request: [SPARK-3970] Remove duplicate removal of local...

2014-10-26 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/2826#issuecomment-60538706 Ok I'm merging this --- 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-3970] Remove duplicate removal of local...

2014-10-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/2826 --- 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-3970] Remove duplicate removal of local...

2014-10-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/2826#issuecomment-60014083 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

[GitHub] spark pull request: [SPARK-3970] Remove duplicate removal of local...

2014-10-17 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/2826#issuecomment-59595864 LGTM, other comments @srowen? --- 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-3970] Remove duplicate removal of local...

2014-10-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/2826#issuecomment-59379911 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

[GitHub] spark pull request: [SPARK-3970] Remove duplicate removal of local...

2014-10-16 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/2826#discussion_r18964919 --- Diff: core/src/main/scala/org/apache/spark/storage/DiskBlockManager.scala --- @@ -140,7 +140,6 @@ private[spark] class DiskBlockManager(blockManager:

[GitHub] spark pull request: [SPARK-3970] Remove duplicate removal of local...

2014-10-16 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/2826#discussion_r18966331 --- Diff: core/src/main/scala/org/apache/spark/storage/DiskBlockManager.scala --- @@ -140,7 +140,6 @@ private[spark] class DiskBlockManager(blockManager:

[GitHub] spark pull request: [SPARK-3970] Remove duplicate removal of local...

2014-10-16 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/2826#discussion_r18979151 --- Diff: core/src/main/scala/org/apache/spark/storage/DiskBlockManager.scala --- @@ -140,7 +140,6 @@ private[spark] class DiskBlockManager(blockManager:

[GitHub] spark pull request: [SPARK-3970] Remove duplicate removal of local...

2014-10-16 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/2826#issuecomment-59457743 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

[GitHub] spark pull request: [SPARK-3970] Remove duplicate removal of local...

2014-10-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/2826#issuecomment-59458746 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-3970] Remove duplicate removal of local...

2014-10-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/2826#issuecomment-59459518 [QA tests have started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/21834/consoleFull) for PR 2826 at commit

[GitHub] spark pull request: [SPARK-3970] Remove duplicate removal of local...

2014-10-16 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/2826#discussion_r18999421 --- Diff: core/src/main/scala/org/apache/spark/storage/DiskBlockManager.scala --- @@ -140,7 +140,6 @@ private[spark] class DiskBlockManager(blockManager:

[GitHub] spark pull request: [SPARK-3970] Remove duplicate removal of local...

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

[GitHub] spark pull request: [SPARK-3970] Remove duplicate removal of local...

2014-10-16 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/2826#issuecomment-59463301 [QA tests have finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/21834/consoleFull) for PR 2826 at commit