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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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
23 matches
Mail list logo