Skip to content

Commit

Permalink
feat(db): Soft-remove storage table (matter-labs#982)
Browse files Browse the repository at this point in the history
## What ❔

Soft-removes the `storage` table, so that it's not referenced in any DB
queries. The table itself will be dropped separately after this code
change is deployed on all envs.

## Why ❔

Faster miniblock sealing; less data in Postgres. Also simplifies
snapshot recovery.

## Checklist

- [x] PR title corresponds to the body of PR (we generate changelog
entries from PRs).
- [x] Tests for the changes have been added / updated.
- [x] Documentation comments have been added / updated.
- [x] Code has been formatted via `zk fmt` and `zk lint`.
- [x] Spellcheck has been run via `zk spellcheck`.
- [x] Linkcheck has been run via `zk linkcheck`.
  • Loading branch information
slowli authored Feb 6, 2024
1 parent b68dd8e commit 601f893
Show file tree
Hide file tree
Showing 44 changed files with 803 additions and 708 deletions.
2 changes: 1 addition & 1 deletion core/bin/snapshots_creator/src/tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -197,7 +197,7 @@ async fn prepare_postgres(
create_miniblock(conn, MiniblockNumber(block_number), logs.clone()).await;

let factory_deps = gen_factory_deps(rng, 10);
conn.storage_dal()
conn.factory_deps_dal()
.insert_factory_deps(MiniblockNumber(block_number), &factory_deps)
.await
.unwrap();
Expand Down

This file was deleted.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

83 changes: 0 additions & 83 deletions core/lib/dal/src/accounts_dal.rs

This file was deleted.

Loading

0 comments on commit 601f893

Please sign in to comment.