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.

Sign up for GitHub

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

Jump to bottom

Very basic willow-store integration with MemStore only. #2809

Draft
wants to merge 1 commit into
base: willow
Choose a base branch
from
Draft

Very basic willow-store integration with MemStore only. #2809

wants to merge 1 commit into from
+462 −169

Conversation

matheus23
Copy link
Contributor

At the moment, this replaces the memory store in willow with the tree in willow-store + its MemStore.

Next steps:

  • Clean up
    • Move willow-store/willow-rs glue into one sane place:
      • conversions: BlobSeq <-> Path, AuthorisedEntry <-> StoredAuthorizedEntry, QueryRange3d <-> Range3d
      • definitions: StoredAuthorizedEntry, WillowParams
    • Fix spelling (Authorised vs. Authorized is inconsitent at the moment)
    • Make use of NamespaceStore again
  • Create a store based on willow_store::RedbBlobStore instead (this way it's persistent)
  • Think about undoing changes to the iroh-willow memory store? Or perhaps make it use redb in memory mode? TBD.
  • Subscriptions and subscription resumption
    • Need access to lower-level details of willow-store to be able to use the trick of using NodeIds for progress_ids.

@matheus23 matheus23 self-assigned this Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@matheus23 matheus23

Labels
None yet
Projects
iroh
Status: 🏗 In progress
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@matheus23