Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

repro and fix for #1788 #1853

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

repro and fix for #1788 #1853

wants to merge 3 commits into from

Conversation

Byron
Copy link
Member

@Byron Byron commented Feb 22, 2025

Fixes #1788.

Tasks

  • reproduction
  • fix

Byron added 3 commits January 13, 2025 15:51
…an hold.

Internally, there is a statically allocated vec which holds opened packs and
indices.
When reconciling the disk-state with what's currently loaded, it was possible
to get into a situation where there were more files than we could fit into
the slotmap.

The system now fails loudly with a specific Error so callers can deal with it.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The slotmap turned out to be too small
1 participant