Reconstruct failed actors without sending tasks.#5161
Reconstruct failed actors without sending tasks.#5161stephanie-wang merged 8 commits intoray-project:masterfrom
Conversation
eb7da5e to
54ef76d
Compare
|
Test PASSed. |
|
Test FAILed. |
zhijunfu
left a comment
There was a problem hiding this comment.
looks good to me. thanks
stephanie-wang
left a comment
There was a problem hiding this comment.
Thanks, this looks good!
python/ray/tests/utils.py
Outdated
| raise Exception("Timing out of wait.") | ||
|
|
||
|
|
||
| def wait_for_contition(condition_predictor, |
There was a problem hiding this comment.
| def wait_for_contition(condition_predictor, | |
| def wait_for_condition(condition_predictor, |
python/ray/tests/utils.py
Outdated
| def wait_for_contition(condition_predictor, | ||
| timeout_ms=1000, | ||
| retry_interval_ms=100): | ||
| """A helper function that wait until a conition is met. |
There was a problem hiding this comment.
| """A helper function that wait until a conition is met. | |
| """A helper function that waits until a condition is met. |
|
Test FAILed. |
|
Test FAILed. |
|
Test FAILed. |
|
Test FAILed. |
|
Test FAILed. |
|
Test PASSed. |
|
Looks like the unit test that was added failed on one of the Travis runs: https://travis-ci.com/ray-project/ray/jobs/215417720. We should increase the timeout for that test. |
|
thanks, increased to 5s |
|
Test PASSed. |
|
Test FAILed. |
|
@stephanie-wang Tests have passed. Can you give a stamp? |
What do these changes do?
Previously, we had to send a task to trigger the reconstruction of a failed actor. This has issues in some cases. For example, an actor that reading data from external DB will never receive tasks. This PR fixes this issue.
Related issue number
Linter
scripts/format.shto lint the changes in this PR.