Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/17144
Ok this LGTM and I merged to master.
I tested this a bunch because in theory, it seems like the check that the
block has been properly re-replicated should / could happen inside the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17144
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/17144
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74089/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17144
**[Test build #74089 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74089/testReport)**
for PR 17144 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17144
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74085/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17144
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/17144
**[Test build #74085 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74085/testReport)**
for PR 17144 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17144
**[Test build #74089 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74089/testReport)**
for PR 17144 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/17144
OK but should the Thread.sleep change be reverted entirely then?
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17144
**[Test build #74085 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74085/testReport)**
for PR 17144 at commit
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/17144
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, or if
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17144
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74058/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17144
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/17144
**[Test build #74058 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74058/testReport)**
for PR 17144 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17144
**[Test build #74058 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/74058/testReport)**
for PR 17144 at commit
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/17144
@kayousterhout sure, I was being doing that flaky 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
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/17144
Also @uncleGen would you mind filing a JIRA for the second failed test case?
---
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 kayousterhout commented on the issue:
https://github.com/apache/spark/pull/17144
cc @shubhamchopra who wrote the original code and @joshrosen who did the
main review
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/17144
I'm not really the right person to review this code, but that being said,
I'm not crazy about this fix, because 1s is kind of a long time to consistently
wait. It's better for tests to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17144
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/17144
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73833/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17144
**[Test build #73833 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73833/testReport)**
for PR 17144 at commit
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/17144
cc @kayousterhout
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17144
**[Test build #73833 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73833/testReport)**
for PR 17144 at commit
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/17144
test crash. retest this please.
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17144
Merged build finished. Test FAILed.
---
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/17144
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73814/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17144
**[Test build #73814 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73814/testReport)**
for PR 17144 at commit
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/17144
one more flaky test? `org.apache.spark.streaming.CheckpointSuite.recovery
with map and reduceByKey operations` I will check it later. retest this please.
---
If your project is set up for it, you
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17144
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73800/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17144
Merged build finished. Test FAILed.
---
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/17144
**[Test build #73800 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73800/testReport)**
for PR 17144 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17144
**[Test build #73800 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73800/testReport)**
for PR 17144 at commit
33 matches
Mail list logo