search

onecodex / mmap-bitvec

Rust implementation of a mmap-based bit vector
MIT License
6 stars 0 forks source link

File Headers? #4

Closed bovee closed 7 years ago

bovee commented 7 years ago

We should optionally (?) allow a (very small?) header with version number.

boydgreenfield commented 7 years ago

@bovee I'd recommend we include the padding by default, use the same size as the Nim one (8 bytes?), and have an explicit opt-out for skipping the header on file creation. All indexing should of course be outside the header area. Header checking (raises an error if fails a version check) should probably be optional but also on by default.

Thoughts? The issue is it'll be a huge PITA to add the padding later even if we don't do any checking on the header in the first version of this lib.