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.
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
feat(evm): genesis import&export for events and confirmedEventQueue #1408
Conversation
utils/queuer.go
Outdated
ExportState() (state QueueState) | ||
ImportState(state QueueState, validator ...func(QueueState) error) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd rather have this in a separate interface (if it's needed at all)
To me, exporting and importing state is not essential to the behaviour of a queue 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll just remove them from the interface.
utils/queuer.go
Outdated
} | ||
|
||
// ImportState imports the given queue state into the kv store | ||
func (q GeneralKVQueue) ImportState(state QueueState, validator ...func(QueueState) error) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
func (q GeneralKVQueue) ImportState(state QueueState, validator ...func(QueueState) error) { | |
func (q GeneralKVQueue) ImportState(state QueueState, validate ...func(QueueState) error) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
on a related note, we do you pass in the state and the validate function instead of passing in validated state?
d585aba
to
88a130e
Compare
Description
Todos
Steps to Test
Expected Behaviour
Other Notes