search

tonarino / actor

A minimalist actor framework aiming for high performance and simplicity.
MIT License
40 stars 6 forks source link

Remove metrics #35

Closed skywhale closed 3 years ago

skywhale commented 3 years ago

Tentatively remove metrics from the crate as per https://github.com/tonarino/actor/issues/34

strohel commented 3 years ago

Looks good! Technically, System and SystemHandle could be now merged into one, if desired?

skywhale commented 3 years ago

I'm curious to hear @mcginty's thought on @strohel's comment. To me, even though they happen to share the same data, the distinction of System and SystemHandle still makes sense. SystemHandle is cloned and moved to each actor thread via Context, whereas System remains non-clone-able as the user-facing entry point.