Skip to content

refactor(sequencer, storage): move rocksdb communication to StorageActor - #798

Open
Arjentix wants to merge 2 commits into
devfrom
arjentix/continue-storage-refactor
Open

refactor(sequencer, storage): move rocksdb communication to StorageActor#798
Arjentix wants to merge 2 commits into
devfrom
arjentix/continue-storage-refactor

Conversation

@Arjentix

Copy link
Copy Markdown
Collaborator

🎯 Purpose

This is a big refactor of our storage. This PR reimplements rocksdb communication from scratch and moves it to StorageActor. Also some protocol types where rethinked.

⚙️ Approach

  • Added actor feature to actor crates which helps dependent actors not to build extra actor and it's dependencies when they need just trait and protocol
  • Transaction index is moved from SequencerCore to StorageActor
  • Rocksdb communication is split into different parts:
    • db.rs: agnostic wrapper around rocksdb crate
    • entities.rs: encoding entities (ex. cells)
    • encoding.rs: encoding utilities
    • actor.rs: high-level management of db
  • Some protocol types for StorageActor where refactored and cleaned
  • Completely removed storage/src/sequencer/ module
  • All tests were ported and added new ones

🧪 How to Test

cargo nextest run --all-features -p sequencer_storage_actor

🔗 Dependencies

None

🔜 Future Work

  • Decide what we do with Indexer
  • And then refactor indexer part of storage crate
  • After that remove storage crate entirely

📋 PR Completion Checklist

  • Complete PR description
  • Implement the core functionality
  • Add/update tests
  • Add/update documentation and inline comments

@Arjentix Arjentix self-assigned this Aug 26, 2026
@Arjentix Arjentix added the priority:medium Normal priority label Aug 26, 2026
@Arjentix
Arjentix force-pushed the arjentix/continue-storage-refactor branch from 6b1135b to 2907759 Compare August 26, 2026 21:21
Comment thread lez/sequencer/actors/storage/src/actor/db.rs Outdated
Comment thread lez/sequencer/actors/storage/src/actor.rs Outdated
Comment thread .deny.toml Outdated
@Arjentix
Arjentix force-pushed the arjentix/continue-storage-refactor branch from 2907759 to 4b2dbdf Compare September 1, 2026 20:16
@Arjentix
Arjentix force-pushed the arjentix/continue-storage-refactor branch from 4b2dbdf to 389b761 Compare September 1, 2026 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority:medium Normal priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants