Skip to content

test(perf): add rdev persistence baseline - #54

Draft
EitanWong wants to merge 2 commits into
mainfrom
perf/53-rdev-baseline-t-f44014f8
Draft

EitanWong wants to merge 2 commits into
mainfrom
perf/53-rdev-baseline-t-f44014f8

Conversation

@EitanWong

@EitanWong EitanWong commented Sep 30, 2026 •

Copy link
Copy Markdown
Owner

Closes #53

Scope

Kanban eng/t_f44014f8: measured baseline and reproducible diagnostic tests/reports only. No production implementation change, migration, deployment, public binary release or claimed optimization GREEN. Draft evidence handoff for the existing independent diagnostic reviewer; do not merge as a production optimization.

Evidence

  • Baseline source9af806f; synthetic seed20260930,1k/10k/20k audit ×20/40 synthetic endpoint/session clients,three repeats.
  • 18primary scenarios1080polls;1360per-persist records including separate frozen-budget/profile windows;45raw benchmark rows.
  • Real nonpreview poll renews lease -> SaveFrom -> full-file history rewrite: one-client163239->1603242B/persist,three REDs against2x growth guard.
  • SaveFrom B/op growth~9.685x;20k/20active handler medianp95=498.895ms vs200ms;stateMu113.72aggregate waitingseconds(91.61%block delay). Audit-only and event-only controls, CPU/heap/block/mutex call stacks retained.
  • Exact supplied ops CSV window21samples/599.842782s: write_bytes delta23150555136B,mean38594371.443B/s,not uptime-derived~45MB/s. No production state/profile copied.

Verification

  • Repository bash scripts/check.sh: actualexit0.
  • go test -race -count=1 ./...: actualexit0.
  • Tagged fixture/matrixrace: actualexit0.
  • Reproducible runner and summarizer: actualexit0;opt-in performanceRED exits1 deliberately retained. Exact report reproduction/count assertions pass.
  • Gitbranchpolicy fixed via realissue53 andperf/53-rdev-baseline-t-f44014f8;old invalidbranch keptforaudit. No policy bypass/historyrewrite.
  • Reports/raw commands/env/exit andCSV/JSON:docs/performance/rdev-baseline-20260930. Synthetic .pprof originals remain in durable localartifact,notgit.

Explicit limitations and CEO decision

Claude analysis requested twice exited124 withaborted_streaming,zeromodelUsage,noeffectiveanalysis/actualmodelreceipt. At2026-09-30 16:04 BeijingLucky/CEO authorized thisBASELINE-ONLY handoff using verifieddata+perf-authored analysis;this isNOTdual-enginePASS. Existingreviewer t_1b876885 independentlyjudgesdiagnostics.

The decision doesnot relaxCoderactualCodex+Claude,finalsameSHAQA/Reviewer/CI,700k/1.4mgrowth,>=60minbaseline/candidatesoak,MCP/managedhostACK/RTT,Windowslivecompatibility,durability/migration/rollbackorreleasegates. Nohealth200/submit202 iscalledhostACK. Nooptimized candidate or deployment claim.

Rollback/safety

Production source unchanged;taggeddiagnostics excludedfromdefaultCI. No realsecret,usercontent,snapshot,binary or production memoryprofile committed. The statistics useprocess-levelIOandnetstatefilesizegrowth;neitherfsyncproofnortruechanged-recordbytes. No deletionofaudit/security weakening/forcedGC.

Measured on 9af806f with synthetic isolated audit/event history. Nonpreview lease polls full-rewrite cold history and block on stateMu; preserve real numeric evidence, strict opt-in REDs and unchanged production code. Claude analysis timed out twice and remains explicitly blocked, not a claimed review.
@coderabbitai

coderabbitai Bot commented Sep 30, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

test(perf): add isolated rdev persistence baseline

1 participant