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

irmin-tezos: simplify the way we serialise nodes #2014

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

samoht
Copy link
Member

@samoht samoht commented Jul 24, 2022

The awkward order is not necessary anymore since irmin 1.4 and no
such order exists in the wild.

@samoht
Copy link
Member Author

samoht commented Jul 25, 2022

Added a commit to optimise the serialisation of inodes of size 1

@samoht samoht added the type/performance Rlated to performance label Aug 4, 2022
@samoht
Copy link
Member Author

samoht commented Aug 4, 2022

This will need to be forward-ported to lib_context: https://gitlab.com/tezos/tezos/-/blob/master/src/lib_context/encoding/context.ml

The awkward order is not necessary anymore since irmin 1.4 and no
such order exists in the wild.
@samoht samoht added the no-changelog-needed No changelog is needed here label Aug 4, 2022
metanivek added a commit to metanivek/tezos-mirror that referenced this pull request Aug 16, 2022
metanivek added a commit to metanivek/tezos-mirror that referenced this pull request Aug 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog-needed No changelog is needed here type/performance Rlated to performance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant