Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/22623
Merging to master / 2.4.
---
-
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/22623
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97179/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22623
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/22623
**[Test build #97179 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97179/testReport)**
for PR 22623 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22623
**[Test build #97179 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97179/testReport)**
for PR 22623 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22623
**[Test build #4365 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4365/testReport)**
for PR 22623 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22623
**[Test build #4362 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4362/testReport)**
for PR 22623 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22623
**[Test build #4362 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4362/testReport)**
for PR 22623 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22623
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97143/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22623
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/22623
**[Test build #97143 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97143/testReport)**
for PR 22623 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22623
**[Test build #97143 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97143/testReport)**
for PR 22623 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22623
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97134/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22623
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/22623
**[Test build #97134 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97134/testReport)**
for PR 22623 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22623
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/22623
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97133/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22623
**[Test build #97133 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97133/testReport)**
for PR 22623 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22623
**[Test build #97134 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97134/testReport)**
for PR 22623 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22623
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97132/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22623
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/22623
**[Test build #97132 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97132/testReport)**
for PR 22623 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22623
**[Test build #97133 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97133/testReport)**
for PR 22623 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22623
**[Test build #97132 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97132/testReport)**
for PR 22623 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22623
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97127/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22623
**[Test build #97127 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97127/testReport)**
for PR 22623 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22623
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/22623
**[Test build #97127 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97127/testReport)**
for PR 22623 at commit
Github user devaraj-kavali commented on the issue:
https://github.com/apache/spark/pull/22623
Yes, you are right, `exitedCleanly` is `false` even when the expected
exception is thrown.
---
-
To unsubscribe, e-mail:
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22623
Just to check my understanding, `exitedCleanly` is `false` even when the
expected exception is thrown? OK that makes sense.
---
Github user devaraj-kavali commented on the issue:
https://github.com/apache/spark/pull/22623
This change is to avoid the expected exception is being thrown from the
thread and getting printed in the test log.
---
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22623
That's checked below. Why this change?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22623
**[Test build #4358 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4358/testReport)**
for PR 22623 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22623
**[Test build #4358 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4358/testReport)**
for PR 22623 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22623
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/22623
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97021/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22623
**[Test build #97021 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97021/testReport)**
for PR 22623 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22623
**[Test build #97021 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97021/testReport)**
for PR 22623 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22623
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/22623
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97003/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22623
**[Test build #97003 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97003/testReport)**
for PR 22623 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22623
**[Test build #97003 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97003/testReport)**
for PR 22623 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/22623
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/22623
Looking at the 2.3 code, it might be the right thing to just remove the
handler for `SparkException` in `SparkSubmit.main`, and let it propagate.
```
try {
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22623
Yeah, that's the issue. Maybe change `printErrorAndExit` to `logError` the
exception in this case. It's noisier to be sure but I think losing the stack
trace here is significant. @vanzin I think you
Github user devaraj-kavali commented on the issue:
https://github.com/apache/spark/pull/22623
Thanks @srowen for looking into this.
> ThreadUtils.scala
```
case NonFatal(t) if !t.isInstanceOf[TimeoutException] =>
throw new
46 matches
Mail list logo
