Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11794#issuecomment-198507761
**[Test build #53543 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53543/consoleFull)**
for PR 11794 at commit
[`108edc8`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11794#issuecomment-198535076
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/11794#discussion_r56697868
--- Diff:
core/src/main/java/org/apache/spark/shuffle/sort/ShuffleExternalSorter.java ---
@@ -320,7 +320,15 @@ private void growPointerArrayIfNecessary() thro
Github user sitalkedia commented on a diff in the pull request:
https://github.com/apache/spark/pull/11794#discussion_r56696662
--- Diff:
core/src/main/java/org/apache/spark/shuffle/sort/ShuffleExternalSorter.java ---
@@ -320,7 +320,15 @@ private void growPointerArrayIfNecessary()
Github user sitalkedia commented on a diff in the pull request:
https://github.com/apache/spark/pull/11794#discussion_r56703069
--- Diff:
core/src/main/java/org/apache/spark/shuffle/sort/ShuffleExternalSorter.java ---
@@ -320,7 +320,15 @@ private void growPointerArrayIfNecessary()
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/11794#discussion_r56690434
--- Diff:
core/src/main/java/org/apache/spark/util/collection/unsafe/sort/UnsafeExternalSorter.java
---
@@ -293,7 +293,15 @@ private void growPointerArrayIfN
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11794
---
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 ena
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11794#issuecomment-198459649
**[Test build #53543 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53543/consoleFull)**
for PR 11794 at commit
[`108edc8`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11794#issuecomment-198534785
**[Test build #53550 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53550/consoleFull)**
for PR 11794 at commit
[`4db3880`](https://g
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/11794#issuecomment-198493534
LGTM
---
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 a
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11794#issuecomment-198079470
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 p
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11794#issuecomment-198488870
**[Test build #53550 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53550/consoleFull)**
for PR 11794 at commit
[`4db3880`](https://gi
GitHub user sitalkedia opened a pull request:
https://github.com/apache/spark/pull/11794
[SPARK-13958]Executor OOM due to unbounded growth of pointer array inâ¦
## What changes were proposed in this pull request?
This change fixes the executor OOM which was recently introdu
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11794#issuecomment-198508305
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/11794#discussion_r56690412
--- Diff:
core/src/main/java/org/apache/spark/shuffle/sort/ShuffleExternalSorter.java ---
@@ -320,7 +320,15 @@ private void growPointerArrayIfNecessary() thro
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/11794#issuecomment-198517715
Merging this into master and 1.6, 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 project
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/11794#issuecomment-198294239
CC @davies
---
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
e
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11794#issuecomment-198508299
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11794#issuecomment-198535070
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 projec
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/11794#issuecomment-198459176
Jenkins, 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 f
20 matches
Mail list logo