-
Notifications
You must be signed in to change notification settings - Fork 99
Labels
network transactionsRelates to the network transactions featureRelates to the network transactions feature
Milestone
Description
There are a series of review comments to address after the merge: #1435
Also fom comment:
I also think this structure is probably half-way to where we'd need to get to eventually. The main things to address in follow-ups:
- How we segment the actor state - the current structure does not seem clean to me - but changing it now will probably result in quite a bit of refactoring.
- How we handle event routing in the coordinator. I think this may have significant performance implications - but also, we should confirm if that's the case first.
- How to handle actor failures - e.g., under what circumstances we may want to respawn failed actors.
The first point is probably also good to address in combination with introducing lazy account loading as both of these will have impact on how the actor state is structured.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
network transactionsRelates to the network transactions featureRelates to the network transactions feature