Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21424
I left the JIRA resolved as "Won't fix" given the discussion above.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.a
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/21424
> Broadcast is special because it's run on driver side, so
SparkOutOfMemoryError is same as OutOfMemoryError, we need to kill the driver
anyway.
Thanks a lot for deep explanation, I will
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21424
That's a good point!
According to the document of `SparkOutOfMemoryError`, Spark should not kill
the executor if `SparkOutOfMemoryError` is thrown. Broadcast is special because
it's run o
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/21424
@cloud-fan
> I also found that we may throw OOM
My previous understanding is that Spark throw `SparkOutOfMemoryError` when
expect there's no memory -- such expectation is from Spark s
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21424
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21424
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21424
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91203/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21424
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91204/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21424
**[Test build #91203 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91203/testReport)**
for PR 21424 at commit
[`8f224fb`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21424
**[Test build #91204 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91204/testReport)**
for PR 21424 at commit
[`3a9669c`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21424
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3621/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21424
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21424
**[Test build #91204 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91204/testReport)**
for PR 21424 at commit
[`3a9669c`](https://github.com/apache/spark/commit/3a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21424
**[Test build #91203 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91203/testReport)**
for PR 21424 at commit
[`8f224fb`](https://github.com/apache/spark/commit/8f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21424
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91105/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21424
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21424
**[Test build #91105 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91105/testReport)**
for PR 21424 at commit
[`aa10470`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21424
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3552/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21424
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21424
**[Test build #91105 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91105/testReport)**
for PR 21424 at commit
[`aa10470`](https://github.com/apache/spark/commit/aa
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/21424
cc @cloud-fan @JoshRosen
Would you please help take a look at this when you have time ?
---
-
To unsubscribe, e-mail: revi
21 matches
Mail list logo