Github user SaintBacchus commented on the issue:
https://github.com/apache/spark/pull/14530
I will re-run this case, and dig into why the executor will double register.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/14530
Looks fine (small style nit but not worth another review round). I'm a
little curious about why it happens - I thought that the driver would kill
executors that failed a heartbeat, but that's a
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14530
As to the actual logic of the change, I don't feel that confident about
reviewing it. Suggest looking at 'blame' output here to see who might have
written it?
---
If your project is set up for it,
Github user SaintBacchus commented on the issue:
https://github.com/apache/spark/pull/14530
\cc @srowen
---
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,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14530
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 project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14530
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63355/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14530
**[Test build #63355 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63355/consoleFull)**
for PR 14530 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14530
**[Test build #63355 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63355/consoleFull)**
for PR 14530 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14530
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63335/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14530
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 project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14530
**[Test build #63335 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63335/consoleFull)**
for PR 14530 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14530
**[Test build #63335 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63335/consoleFull)**
for PR 14530 at commit
12 matches
Mail list logo
