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

feat(rln-relay): resume onchain sync from persisted tree db #1805

Merged
merged 2 commits into from
Jun 19, 2023

Conversation

rymnc
Copy link
Contributor

@rymnc rymnc commented Jun 16, 2023

Description

This feature makes use of the new ffi api's set_metadata and get_metadata introduced in #1803. This allows us to insert the block number we sync
to, and to resume sync when we re-use the database. reduces complexity and improves node startup time significantly

Changes

  • Use setMetadata in handleEvents
  • Use getMetadata in init for the group manager

Issue

closes #1772

@rymnc rymnc self-assigned this Jun 16, 2023
@rymnc rymnc marked this pull request as ready for review June 16, 2023 07:00
@SionoiS
Copy link
Contributor

SionoiS commented Jun 16, 2023

LGTM!

Feels good to look at some Rust code again :P

@rymnc
Copy link
Contributor Author

rymnc commented Jun 16, 2023

@SionoiS would you like me to add you as a reviewer for some of the rust pr's later if you've got some extra time?

@SionoiS
Copy link
Contributor

SionoiS commented Jun 16, 2023

@SionoiS would you like me to add you as a reviewer for some of the rust pr's later if you've got some extra time?

@rymnc No problem!

Copy link
Collaborator

@Ivansete-status Ivansete-status left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks!

@rymnc rymnc merged commit bbded9e into master Jun 19, 2023
@rymnc rymnc deleted the resync-from-last-point branch June 19, 2023 09:51
SionoiS pushed a commit that referenced this pull request Jun 19, 2023
* feat(rln-relay): resume onchain sync from persisted tree db

* chore(rln-relay): bump zerokit
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.

feat(rln-relay): allow passing in a path to the tree db
3 participants