… for node ID → revision number lookups, instead on linear scan in a revlog.
- Group Reviewers
- rHG9eb07ab3f2d4: rhg: use persistent nodemap when available
I would add a comment explaining why it's ok for the docket to not exist, even if it's kind of obvious.
32 bit systems will have other issues before running into this one, I don't think it's worth it.
I think we probably want to be able to disable mmap even in Rust contexts, although that will be done in a followup, since we need to have access to the config (storage.revlog.nodemap.mmap at the time of writing).
I think the Python code has a bug, and that the idea is to silently ignore the missing file.
Please add a docstring
I think this comment should be adjusted now that the optimization has come
I'd add a small header line to say "test partial node prefix" or something similar.