Github user viktorklang commented on the issue:
https://github.com/apache/spark/pull/21342
Scala 2.13.0-M5 and forward has improved handling of InterruptedExceptions
and fatal errors should be propagated, I'd welcome feedback on the new logic
when possible. Thank you!
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/21342
Thanks for merging !
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21342
Thanks! Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user JoshRosen commented on the issue:
https://github.com/apache/spark/pull/21342
Updated changes LGTM. Thanks for working on this!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91039/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21342
**[Test build #91039 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91039/testReport)**
for PR 21342 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
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/3508/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21342
**[Test build #91039 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91039/testReport)**
for PR 21342 at commit
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/21342
Thanks a gain for comments !
I updated this pr and added small defensive logic in
SparkUncaughtExceptionHandler.scala.
Please take another look ~
---
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/21342
https://issues.scala-lang.org/browse/SI-9554?orig=1 is still "OPEN", not
sure which scala version can fix this bug.
---
-
To
Github user JoshRosen commented on the issue:
https://github.com/apache/spark/pull/21342
Thanks for the updates. The net change / scope of changes have been
significantly reduced here, so I feel that this change is a lot less risky now.
I left only one nitpicky comment at
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21342
LGTM, which scala version has fixed this bug?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90853/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21342
**[Test build #90853 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90853/testReport)**
for PR 21342 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
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/3384/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21342
**[Test build #90853 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90853/testReport)**
for PR 21342 at commit
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/21342
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
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/3380/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90847/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21342
**[Test build #90847 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90847/testReport)**
for PR 21342 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/21342
@kiszk @cloud-fan
Thanks a lot for comments.
I tested manually and found that the bug exists in all fatal throwable.
In current change, I catch all fatal throwable and wrap in
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/21342
I will update the pr title if the change is on the right
direction.
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21342
**[Test build #90847 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90847/testReport)**
for PR 21342 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90846/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21342
**[Test build #90846 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90846/testReport)**
for PR 21342 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
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/3379/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21342
**[Test build #90846 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90846/testReport)**
for PR 21342 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21342
How about we create a special Exception class to wrap OOM and
InterupttedException, and say it's only used to work around the scala bug?
---
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/21342
Basically, I agree with this two-step direction. However, I think that it
is too risky to simply throw and catch `SparkException`. This is because other
`SparkException` may be caught in step 2. For
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/21342
Thanks a lot for looking into this.
The issue is that, sometimes user would configure
`spark.sql.broadcastTimeout` as bigger value, because the `relationFuture` in
Github user JoshRosen commented on the issue:
https://github.com/apache/spark/pull/21342
I'm also in favor of delaying for a couple of days for more detailed review
because historically I think these types of changes have been high risk. The
risk calculus might be a bit different if
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21342
Could we hold this PR? This change sounds risky. `OutOfMemoryError` is not
issued by Spark. If we eat it, it could cause serious outcomes. cc @zsxwing
@JoshRosen
---
Github user sameeragarwal commented on the issue:
https://github.com/apache/spark/pull/21342
LGTM, Thanks! I've seen variations of this bug in the past as well and am
not aware of a better way to fix this issue.
```scala
scala> Await.result(Future(throw new
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/21342
The change looks good.
Is it better to add a comment that describes why we throw `SparkException`
instead of `OOM` with a reference to `scala/bug#9554`. In the future, we would
throw OOM if scala
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/21342
cc @sameeragarwal @hvanhovell @cloud-fan @jiangxb1987
Please take a look at this when you have time.
---
-
To
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90676/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21342
**[Test build #90676 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90676/testReport)**
for PR 21342 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90675/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21342
**[Test build #90675 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90675/testReport)**
for PR 21342 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
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/3256/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21342
**[Test build #90676 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90676/testReport)**
for PR 21342 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
Test FAILed.
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/3255/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21342
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21342
**[Test build #90675 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90675/testReport)**
for PR 21342 at commit
55 matches
Mail list logo