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

Support complex lock strategies in atomic execution protocol #149

Open
adlrocha opened this issue Mar 17, 2022 · 0 comments
Open

Support complex lock strategies in atomic execution protocol #149

adlrocha opened this issue Mar 17, 2022 · 0 comments

Comments

@adlrocha
Copy link
Collaborator

adlrocha commented Mar 17, 2022

In the current implementation, all messages of the execution should affect the same method, and we don't give a way to easily lock different parts of the state of an actor according to the params of each of this messages, we just look to the first message and lock according (and execute) according to the method and params it has. We should generalize this approach to allow the locking and execution of any number of messages and params in an actor.

// FIXME: We are using here the method and params from the first message.
// This works because we are currently using a really simple LockableActor,
// this will need to be further generalized when we introduce new actors.
lparams, err := atomic.WrapSerializedParams(msgs[0].Method, msgs[0].Params)

// FIXME: Disregarding params to lock for now

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant