Skip to content
This repository has been archived by the owner on Mar 10, 2023. It is now read-only.

Allow registering LuaActor to actix register #12

Open
poga opened this issue Sep 18, 2018 · 3 comments
Open

Allow registering LuaActor to actix register #12

poga opened this issue Sep 18, 2018 · 3 comments
Labels
enhancement New feature or request

Comments

@poga
Copy link
Owner

poga commented Sep 18, 2018

Is your feature request related to a problem? Please describe.
Currently, you can't register LuaActor to System or Arbiter since it does not implement SystemService. In order to implement SystemService, LuaActor must impl Default.

The problem is, how do we implement Default for LuaActor? I think the reason actix requires Default for services is that it need to restart them when crashed?

Describe the solution you'd like
Figure out how to implement Default for LuaActor. Then implement SystemService, ArbiterService, and Supervised for it.

Describe alternatives you've considered
Not sure, need to understand more detail on actix's registry.

@poga poga added the enhancement New feature or request label Sep 18, 2018
@poga poga changed the title Allow registering LuaActor to System/Arbiter Allow registering LuaActor as a SystemService Dec 12, 2018
@poga
Copy link
Owner Author

poga commented Dec 14, 2018

https://github.com/Chris-Ricketts/actix-broker might be a potential solution

@poga
Copy link
Owner Author

poga commented Dec 14, 2018

Also, only one actor per type can be registered to the registry, since LuaActor is a concrete type, we need to work around that.

@poga poga changed the title Allow registering LuaActor as a SystemService Allow registering LuaActor to actix register Dec 14, 2018
@poga
Copy link
Owner Author

poga commented Jan 19, 2019

Actix 0.7.10 provides WeakAddr<A> for weak referencing an actor, which can be used to provide custom registry.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant