Skip to content

Reduce recovery memo storage - #971

Merged
odvcencio merged 2 commits into
mainfrom
codex/perf-recovery-memo-tier
Aug 24, 2026
Merged

Reduce recovery memo storage#971
odvcencio merged 2 commits into
mainfrom
codex/perf-recovery-memo-tier

Conversation

@odvcencio

Copy link
Copy Markdown
Owner

Summary

  • Preserve benchmark-only commit 8e8f18b8.
  • Reduce active recovery memo storage from 6 MiB to 3 MiB.
  • Bound combined backing storage at 3.375 MiB.

Evidence

  • The durable benchmark reduces bytes per operation by 20.79%, with p<0.001.
  • Time and allocation results are inconclusive.
  • Paired RSS runs reduce maximum usage by 11,520 KiB and 4,160 KiB.
  • Focused C# correctness, locked-C parity, and cache-contract gates pass.
  • The primary trio is order-sensitive and inconclusive.
  • The 20-seed randomized recovery benchmark completes.

Advances #454

- Add benchmark for 32 KiB C# file with edit marker removed.
- Disable admission candidate route to match existing recovery tests.
- Capture performance metrics for the parser recovery path.

Buckley-Change-Hash: sha256:27ebc4d6d1252f80b5d5e996c8ec1eaf09a1d4a463be6d0303036ce7c577e228
Buckley-Change-Stats: files=1 insertions=22 deletions=0 binaries=0
- Reduce the active memo storage from 6 MiB to 3 MiB.
- Bound the combined backing storage at 3.375 MiB.
- Record a 20.79% bytes-per-operation reduction.
- Classify time and allocation results as inconclusive.
- Preserve focused C# correctness and locked-C parity.

Buckley-Change-Hash: sha256:5fad84815ca75725b8ed7c63fddbae4114ddaca31cb63cd01ee41952aa52b2ed
Buckley-Change-Stats: files=4 insertions=107 deletions=4 binaries=0
@odvcencio
odvcencio merged commit b76b653 into main Aug 24, 2026
49 checks passed
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.

1 participant