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-pack: Fix failure when computing summary in tree benchmark #1968
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1968 +/- ##
=======================================
Coverage 63.95% 63.95%
=======================================
Files 130 130
Lines 15479 15479
=======================================
Hits 9900 9900
Misses 5579 5579
📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more |
Yeah, it's very likely that there is some deeper issue but this at least gets us some usable output from the tree benchmark. |
That's a great insight but I am not convinced by the fix :-) Could you try to trace when an empty tree is generated/written on disk? This is never supposed to happen. I also think we should remove that option and always fail hard when this happens as this is almost always due to a bug - unless I am missing something. |
Right after closing this I figured out the actual issue - sorry for the noise! It looks like the store was being initialized with an empty tree in This PR now removes that initial empty commit and everything seems to be working - luckily this means this issue does not come from the tezos trace. |
Saving an empty root actually seems a valid use case. I would just make it a no-op so that empty trees are actually never persisted on disk. |
Tracked in #1969 ; your fix is great for now :-) |
This PR makes it so
./bench/irmin-pack/tree.exe
runs successfully and is able to produce a summary.Fixes #1965