Skip to content
This repository has been archived by the owner on May 23, 2022. It is now read-only.

Use a KV store to map heights between optimint blocks #39

Closed
1 task
evan-forbes opened this issue Jan 13, 2022 · 1 comment
Closed
1 task

Use a KV store to map heights between optimint blocks #39

evan-forbes opened this issue Jan 13, 2022 · 1 comment
Labels
good first issue Good for newcomers

Comments

@evan-forbes
Copy link
Member

Currently, we are using a json map to keep track of optimint heights and their corresponding celestia heights. As discussed in #17, we want to use something more robust, like a standard KV store.

This will better position the DALC to keep optimint platform agnostic, and offload some responsibility as well.

  • Replace HeightMapper's in memory map with a KV store
@evan-forbes evan-forbes moved this to TODO in Celestia Node Jan 13, 2022
@evan-forbes evan-forbes mentioned this issue Jan 13, 2022
Merged
@tzdybal tzdybal added the good first issue Good for newcomers label Jan 21, 2022
@evan-forbes
Copy link
Member Author

can close as height mapping functionality has been removed
#58

Repository owner moved this from TODO to Done in Celestia Node Mar 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
good first issue Good for newcomers
Projects
No open projects
Celestia Node
Archived in project
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tzdybal @evan-forbes