Skip to content

Conversation

jovany-wang
Copy link
Contributor

What do these changes do?

  1. fork() API will fork an actor handle explicitly that we could use it to do a other Ray.calls.
  2. For serializer, we implicitly do fork when serializing recursively.

For example:

class MyObject {
private RayActor actor;
}

or

class MyObject {
private MyObject1 obj1;
}

class MyObject1 {
private RayActor actor;
}

If the actor object actor is a field of another object obj(MyObject), when we pass obj by Ray.call, the obj.actor's handleId will be changed to another id.

Related issue number

N/A

@jovany-wang jovany-wang requested a review from raulchen January 4, 2019 05:50
@AmplabJenkins
Copy link

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

@raulchen raulchen changed the title [Java] Add fork() API for serializer [Java] Allow actor handle to be serialized without forking Jan 4, 2019
@AmplabJenkins
Copy link

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

@AmplabJenkins
Copy link

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

@raulchen raulchen merged commit 692fdc6 into ray-project:master Jan 5, 2019
@jovany-wang jovany-wang deleted the fork-actor-handle-id branch January 17, 2019 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants