perf(shared-fs): trim redundant batch result assembly - #345
peerbit-org wants to merge 1 commit into
Conversation
|
First-attempt raw-log audit for exact head
Neither original first-attempt failure stack is exposed by the ordinary reporter. Nearby snapshot-publication stderr is context, not established cause. These are old-cohort jobs, not evidence against the upcoming upstream release. No job was rerun and no timeout/assertion changed. Keeping this draft held. The isolated next-cohort integration branch includes the existing strict first-failure reporter, the durable-readiness upgrade, trust-cache fence and this small cleanup. It will receive only the verified published cohort and fresh strict cross-platform validation; it does not overwrite the original checkout or held PR worktrees. No auto-merge or release. |
Summary
Remove result-only work from each applied
writeBatchedit. The new version includes every captured head as a parent, so the old-head filter has no surviving entries. Pass the existingFileVersiondirectly to the formatter, aswriteFilealready does.This removes two temporary row objects, two parent-list JSON decodes, a Set and a redundant old-head scan per applied edit. No dependency, trust, live namespace lookup, publication order, manifest, persisted receipt or post-version chunk-presence repair changes. No end-to-end speedup claim.
Validation
9472e470b864a8cbd7734320e5c64d65f4d14f71fb808a4e61b1139ccc91477f.Development fixture mistakes (history enumeration order and test-only typing) were corrected before final validation; no product timeout or retry policy was changed. The first full strict run passed; it was not rerun.
This is based on master
b2c9d300, separate from held trust PR #344 and held cleanup/upgrade branches. A separate 16-worker comparison of baseline versus #344 found batching faster in 12/16 matched cases, including counterexamples on baseline; that experiment does not measure this cleanup or clear #344's old strict failures. Keeping this draft pending fresh CI/raw retry audit; no auto-merge or release.