Skip to content

Saffron: storage prototype #3224

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

Open
wants to merge 19 commits into
base: master
Choose a base branch
from
Open

Saffron: storage prototype #3224

wants to merge 19 commits into from

Conversation

anne-laure-s
Copy link
Contributor

This PR adds a basic storage handling for state replicators

volhovm
volhovm previously approved these changes May 12, 2025
Copy link
Member

@volhovm volhovm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure this is better than implementing some kind of in-memory storage that just writes into the file every now and then automatically, but we can start off with this and change along the way. The final solution will be much smarter I think (some kind of on-disk DB with smart caching is necessary I think).

@dannywillems
Copy link
Member

Why do we have these methods in Rust if the state replicator is in OCaml?

@dannywillems
Copy link
Member

Note that you can use different backends already implemented in the Mina codebase.

@anne-laure-s anne-laure-s force-pushed the al/encodings branch 2 times, most recently from fac0dd4 to aae0111 Compare May 29, 2025 17:54
Base automatically changed from al/encodings to master June 4, 2025 17:44
@volhovm volhovm dismissed their stale review June 4, 2025 17:44

The base branch was changed.

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.

3 participants