[GitHub] spark pull request: [SPARK-14911][Core] Fix a potential data race ...

2016-04-27 Thread lw-lin
Github user lw-lin commented on the pull request: https://github.com/apache/spark/pull/12681#issuecomment-214983254 @davies thanks for the review & merging :-) --- 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-14911][Core] Fix a potential data race ...

2016-04-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12681 --- 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-14911][Core] Fix a potential data race ...

2016-04-27 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/12681#issuecomment-214980242 LGTM, Merging this into master, thanks! --- 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-14911][Core] Fix a potential data race ...

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12681#issuecomment-214962374 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-14911][Core] Fix a potential data race ...

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

[GitHub] spark pull request: [SPARK-14911][Core] Fix a potential data race ...

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

[GitHub] spark pull request: [SPARK-14911][Core] Fix a potential data race ...

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

[GitHub] spark pull request: [SPARK-14911][Core] Fix a potential data race ...

2016-04-26 Thread lw-lin
Github user lw-lin commented on the pull request: https://github.com/apache/spark/pull/12681#issuecomment-214939654 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 have

[GitHub] spark pull request: [SPARK-14911][Core] Fix a potential data race ...

2016-04-26 Thread lw-lin
Github user lw-lin commented on the pull request: https://github.com/apache/spark/pull/12681#issuecomment-214753491 Some flaky tests unrelated to this PR Jenkins retest this please --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-14911][Core] Fix a potential data race ...

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

[GitHub] spark pull request: [SPARK-14911][Core] Fix a potential data race ...

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12681#issuecomment-214743759 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-14911][Core] Fix a potential data race ...

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

[GitHub] spark pull request: [SPARK-14911][Core] Fix a potential data race ...

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

[GitHub] spark pull request: [SPARK-14911][Core] Fix a potential data race ...

2016-04-26 Thread lw-lin
Github user lw-lin commented on the pull request: https://github.com/apache/spark/pull/12681#issuecomment-214713457 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 have

[GitHub] spark pull request: [SPARK-14911][Core] Fix a potential data race ...

2016-04-25 Thread lw-lin
Github user lw-lin commented on the pull request: https://github.com/apache/spark/pull/12681#issuecomment-214617805 Some flaky tests not caused by this PR. Jenkins retest this please --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-14911][Core] Fix a potential data race ...

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

[GitHub] spark pull request: [SPARK-14911][Core] Fix a potential data race ...

2016-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12681#issuecomment-214617524 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-14911][Core] Fix a potential data race ...

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

[GitHub] spark pull request: [SPARK-14911][Core] Fix a potential data race ...

2016-04-25 Thread lw-lin
Github user lw-lin commented on the pull request: https://github.com/apache/spark/pull/12681#issuecomment-214605511 @davies (who made the first change) might want to take a look? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark pull request: [SPARK-14911][Core] Fix a potential data race ...

2016-04-25 Thread lw-lin
Github user lw-lin commented on the pull request: https://github.com/apache/spark/pull/12681#issuecomment-214601397 Actually this wouldn't cause any problem and wouldn't fail any test suits **_for now_**, because the read of `acquiredButNotUsed` is guaranteed to see most recent value

[GitHub] spark pull request: [SPARK-14911][Core] Fix a potential data race ...

2016-04-25 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12681#issuecomment-214598957 cc @andrewor14 and @JoshRosen --- 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-14911][Core] Fix a potential data race ...

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

[GitHub] spark pull request: [SPARK-14911][Core] Fix a potential data race ...

2016-04-25 Thread lw-lin
GitHub user lw-lin opened a pull request: https://github.com/apache/spark/pull/12681 [SPARK-14911][Core] Fix a potential data race in TaskMemoryManager ## What changes were proposed in this pull request? [[SPARK-13210][SQL] catch OOM when allocate memory and expand