Fix actor ID collision in local mode #5863

edoakes · 2019-10-08T17:40:27Z

Based off of #5815 from @justinwyang.

Related issue number

Closes #5715

I've run scripts/format.sh to lint the changes in this PR.
I've included any doc changes needed for https://ray.readthedocs.io/en/latest/.
I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failure rates at https://ray-travis-tracker.herokuapp.com/.

Co-Authored-By: Edward Oakes <[email protected]>

AmplabJenkins · 2019-10-08T19:20:19Z

Can one of the admins verify this patch?

AmplabJenkins · 2019-10-08T21:39:03Z

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Ray-PRB/17556/
Test FAILed.

justinwyang and others added 8 commits October 1, 2019 19:38

Fixed local mode actor id

b31d39c

Update python/ray/actor.py

5b78a55

Co-Authored-By: Edward Oakes <[email protected]>

Added hyphen to match comments

a220681

Added tests to test_local_mode

20085f9

Helloworld

5b12897

Better test naming

37d3d24

Merge branch 'master' into local-mode-actor-id

94adefa

lint

8da1328

edoakes requested a review from richardliaw October 8, 2019 17:40

edoakes changed the title ~~Local mode actor~~ Fix actor ID collision in local mode Oct 8, 2019

edoakes mentioned this pull request Oct 8, 2019

Fixed local mode actor id #5815

Closed

richardliaw approved these changes Oct 8, 2019

View reviewed changes

edoakes merged commit 42dd0fa into ray-project:master Oct 8, 2019