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

irmin-pack: remove Io_legacy #1955

Closed
samoht opened this issue Jul 5, 2022 · 2 comments
Closed

irmin-pack: remove Io_legacy #1955

samoht opened this issue Jul 5, 2022 · 2 comments
Labels
area/backend About the backend feature/layered-store Related to the Layered Store type/refactor Code refactoring

Comments

@samoht
Copy link
Member

samoht commented Jul 5, 2022

Io_legacy is used by irmin-pack to manage "atomic-write" stores that store branches. This is not used in production by Tezos so should be easy to refactor at one point.

This is not high priority but that would be nice to have only one Io module ...
@samoht samoht added type/refactor Code refactoring feature/layered-store Related to the Layered Store area/backend About the backend labels Jul 5, 2022
@samoht samoht mentioned this issue Jul 5, 2022
Merged
5 tasks
@adatario
Copy link
Contributor

Seems to be related to #2134. As mentioned in #2125 (comment) Io_legacy is still used by the branch store.

@metanivek
Copy link
Member

Removed in #2268

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/backend About the backend feature/layered-store Related to the Layered Store type/refactor Code refactoring
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@samoht @metanivek @adatario