Skip to content

Conversation

@chhtz
Copy link
Member

@chhtz chhtz commented Nov 10, 2025

Related to #33 @planthaber
This appears to work, but I haven't tested it extensively (and not really benchmarked either), and only on Ubuntu 24, with rock_display_bin (which may not be 100% stable as well)

The index-file could probably also be switched to an mmap.

I'm not sure what happens if

  • multiple users access the same file
  • files get really big

@chhtz chhtz requested a review from planthaber November 10, 2025 16:58
@chhtz
Copy link
Member Author

chhtz commented Nov 10, 2025

MappedStream is maybe not the best name, since it does not "stream". If the index-file can use this as well, then FileStream could be dropped.

I copied and edited FileStream.{hpp,cpp} therefore some comments are wrong, of course.

@chhtz
Copy link
Member Author

chhtz commented Nov 11, 2025

Further TODOs:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants