Uh oh!
There was an error while loading. Please reload this page.
perf(fspy): use sparse Windows shared memory - #525
Conversation
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit:b38ebb3d06
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Uh oh!
There was an error while loading. Please reload this page.
29bdc15 to
9bcba8eCompare9bcba8e to
62be316Compare6df5e61 to
91edb14Compare62be316 to
283d922Compare283d922 to
6691051Compare6691051 to
033dbe6Compare033dbe6 to
079c1afCompare2fb905b to
83d2b75Compare83d2b75 to
f3a5ac6Comparef3a5ac6 to
e7205e0Compare2f24313 to
bc90566Comparee7205e0 to
1161f5bCompare1161f5b to
ddfb5f3Compareddfb5f3 to
13bda67CompareCo-authored-by: GPT-5.6 <gpt-5.6@openai.com> Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
13bda67 to
45c8d2fCompare
Motivation
Avoid allocating a fully backed 4 GiB temporary file for every tracked task on Windows.
Background
The parity backend deliberately retains the previous file-backed resource model. Marking that file sparse allows the logical mapping to remain 4 GiB while physical allocation follows the small prefix actually used by fspy records.
Changes
FSCTL_SET_SPARSEbefore extending the file.