Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Change message InvocationRequest using just the Actor name instead of the entire Actor entity #3

Open
sleipnir opened this issue Jun 24, 2022 · 1 comment
Open

Change message InvocationRequest using just the Actor name instead of the entire Actor entity #3

sleipnir opened this issue Jun 24, 2022 · 1 comment
Assignees
@sleipnir
Labels
enhancement New feature or request protocol Specification of protocol
Milestone
v1.0.0

Comments

@sleipnir
Copy link
Member

Currently, it is necessary to pass the entire Actor Entity in the invocation message, but only the Actor and ActorSystem name are needed so that it is possible to lookup the Actor Process.
Removing the entity would save a few bytes and make the protocol less confusing.
@sleipnir sleipnir added enhancement New feature or request protocol Specification of protocol labels Jun 24, 2022
@sleipnir sleipnir assigned sleipnir and marcellanz and unassigned sleipnir and marcellanz Jun 24, 2022
@marcellanz
Copy link
Member

I agree... are there potential uses? Also, if really needed we could add it again. I think better to strip it down to whats needed that to drag that with us.

@sleipnir sleipnir linked a pull request Jul 11, 2022 that will close this issue
Improvement the Protocol #8
Closed
@sleipnir sleipnir added this to the v1.0.0 milestone Sep 9, 2022
@sleipnir sleipnir removed this from Spawn v0.5.0 Sep 22, 2022
@sleipnir sleipnir moved this to In Progress in Spawn v1.0.0 Sep 22, 2022
@sleipnir sleipnir moved this from In Progress to Archived in Spawn v1.0.0 Aug 7, 2023
@sleipnir sleipnir moved this from Archived to Todo in Spawn v1.0.0 Aug 7, 2023
@sleipnir sleipnir removed the status in Spawn v1.0.0 Oct 13, 2023
@sleipnir sleipnir removed this from Spawn v1.0.0 Dec 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sleipnir sleipnir

Labels
enhancement New feature or request protocol Specification of protocol
Projects
Spawn v1.5.0
Status: Archived
Milestone
v1.0.0
Development

Successfully merging a pull request may close this issue.

Improvement the Protocol eigr/spawn
2 participants
@sleipnir @marcellanz