Skip to content
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

[CX_CLEANUP] - Deprecate and remove Storage trait and usages #2788

Closed
Tracked by #2356
jparr721 opened this issue Mar 15, 2024 · 0 comments · Fixed by #2790
Closed
Tracked by #2356

[CX_CLEANUP] - Deprecate and remove Storage trait and usages #2788

jparr721 opened this issue Mar 15, 2024 · 0 comments · Fixed by #2790

Comments

@jparr721
Copy link
Contributor

What is this task and why do we need to work on it?

We have a lot of old dead code because conensus used be completely state driven, i.e. blocks were state updates, it used to be very explicitly one view at a time, not event driven. One of these artifacts exists as the Storage trait, found here.

As a result of the need to support storing DA, VID diseperals, and later work by sequencer (tracking: #2621), we need to deprecate and remove this type from the code and modernize its name. This issue is preliminarily responsible for only the removal of this type from the code.

What work will need to be done to complete this task?

  • Remove the type from HotShot-Types
  • Remove MemoryStorage
  • Remove AtomicStorage

Are there any other details to include?

No response

What are the acceptance criteria to close this issue?

Remove all instances of the Storage trait and derivative implementations. All existing tests should still pass without needing to change anything in a substantial way.

Branch work will be merged to (if not the default branch)

No response

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

Successfully merging a pull request may close this issue.

1 participant