Skip to content

Cleanup MultiWriterAppStore#1609

Merged
enlight merged 2 commits intomasterfrom
pr1532-followup3
Jan 30, 2020
Merged

Cleanup MultiWriterAppStore#1609
enlight merged 2 commits intomasterfrom
pr1532-followup3

Conversation

@enlight
Copy link
Copy Markdown
Contributor

@enlight enlight commented Jan 30, 2020

MultiWriterAppStore was used to migrate EVM state keys from app.db to evm.db, since all clusters have now been migrated this functionality is no longer necessary, and is in fact incompatible with the changes introduced in PR #1532.

This set of changes cleans out most of the obsolete code from that store:

  • Eliminate writes to the EvmStore from MultiWriterAppStore.
  • Eliminate Get/Has/Set/Delete from EvmStore.
  • Eliminate use of AppStore.SaveEVMStateToIAVL config setting.

MultiWriterAppStore was used to migrate EVM state keys from app.db to
evm.db, since all clusters have now been migrated this functionality is
no longer necessary, and is in fact incompatible with the changes
introduced in PR #1532.

This set of changes cleans out most of the obsolete code from that store:
- Eliminate writes to the EvmStore from MultiWriterAppStore.
- Eliminate Get/Has/Set/Delete from EvmStore.
- Eliminate use of AppStore.SaveEVMStateToIAVL config setting.
@enlight enlight merged commit 0ea1e1e into master Jan 30, 2020
@enlight enlight deleted the pr1532-followup3 branch January 30, 2020 09:43
enlight added a commit that referenced this pull request Mar 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant