[GitHub] spark pull request: [SPARK-12552][Core]Correctly count the driver ...

2016-03-30 Thread GavinGavinNo1
Github user GavinGavinNo1 commented on the pull request: https://github.com/apache/spark/pull/10506#issuecomment-203702270 Thank you for your comment for PR #12054. I think changing app state from WAITING to RUNNING in function completeRecovery. Suppose that some app is WAITING

[GitHub] spark pull request: [SPARK-12552][Core]Correctly count the driver ...

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

[GitHub] spark pull request: [SPARK-12552][Core]Correctly count the driver ...

2016-03-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10506#issuecomment-190629099 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

[GitHub] spark pull request: [SPARK-12552][Core]Correctly count the driver ...

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

[GitHub] spark pull request: [SPARK-12552][Core]Correctly count the driver ...

2016-02-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10506#issuecomment-190579869 **[Test build #52225 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52225/consoleFull)** for PR 10506 at commit

[GitHub] spark pull request: [SPARK-12552][Core]Correctly count the driver ...

2016-02-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10506#issuecomment-190575725 **[Test build #52223 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52223/consoleFull)** for PR 10506 at commit

[GitHub] spark pull request: [SPARK-12552][Core]Correctly count the driver ...

2016-02-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10506#issuecomment-190575732 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-12552][Core]Correctly count the driver ...

2016-02-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10506#issuecomment-190575730 Merged build finished. Test FAILed. --- 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-12552][Core]Correctly count the driver ...

2016-02-29 Thread jerryshao
Github user jerryshao commented on the pull request: https://github.com/apache/spark/pull/10506#issuecomment-190575489 @andrewor14 , would you please review this patch again, it is pending here a long time and I think it is actually a bug here. Thanks a lot. --- If your project is

[GitHub] spark pull request: [SPARK-12552][Core]Correctly count the driver ...

2016-02-29 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10506#issuecomment-190575051 **[Test build #52223 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52223/consoleFull)** for PR 10506 at commit

[GitHub] spark pull request: [SPARK-12552][Core]Correctly count the driver ...

2015-12-30 Thread jerryshao
Github user jerryshao commented on the pull request: https://github.com/apache/spark/pull/10506#issuecomment-167955973 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-12552][Core]Correctly count the driver ...

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

[GitHub] spark pull request: [SPARK-12552][Core]Correctly count the driver ...

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

[GitHub] spark pull request: [SPARK-12552][Core]Correctly count the driver ...

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

[GitHub] spark pull request: [SPARK-12552][Core]Correctly count the driver ...

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

[GitHub] spark pull request: [SPARK-12552][Core]Correctly count the driver ...

2015-12-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10506#issuecomment-167955369 Merged build finished. Test FAILed. --- 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-12552][Core]Correctly count the driver ...

2015-12-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10506#issuecomment-167972801 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

[GitHub] spark pull request: [SPARK-12552][Core]Correctly count the driver ...

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

[GitHub] spark pull request: [SPARK-12552][Core]Correctly count the driver ...

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

[GitHub] spark pull request: [SPARK-12552][Core]Correctly count the driver ...

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

[GitHub] spark pull request: [SPARK-12552][Core]Correctly count the driver ...

2015-12-29 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/10506#issuecomment-167860300 Can you add a unit test? You might have to mock the `completeRecovery` method --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-12552][Core]Correctly count the driver ...

2015-12-29 Thread jerryshao
Github user jerryshao commented on the pull request: https://github.com/apache/spark/pull/10506#issuecomment-167911683 Sure, will do. --- 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-12552][Core]Correctly count the driver ...

2015-12-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10506#issuecomment-167765195 Merged build finished. Test FAILed. --- 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-12552][Core]Correctly count the driver ...

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

[GitHub] spark pull request: [SPARK-12552][Core]Correctly count the driver ...

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

[GitHub] spark pull request: [SPARK-12552][Core]Correctly count the driver ...

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

[GitHub] spark pull request: [SPARK-12552][Core]Correctly count the driver ...

2015-12-29 Thread jerryshao
Github user jerryshao commented on the pull request: https://github.com/apache/spark/pull/10506#issuecomment-167793719 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-12552][Core]Correctly count the driver ...

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