Skip to content

docs(agents): record the 2026-08-27 perf-audit refresh and the open SqlFile fsync cheap fix - #898

Merged
pmrv merged 1 commit into
mainfrom
claude/peaceful-bell-0nndbb
Aug 28, 2026
Merged

docs(agents): record the 2026-08-27 perf-audit refresh and the open SqlFile fsync cheap fix#898
pmrv merged 1 commit into
mainfrom
claude/peaceful-bell-0nndbb

Conversation

@pmrv

@pmrv pmrv commented Aug 28, 2026

Copy link
Copy Markdown
Owner

Weekly AGENTS.md/DEVELOPING.md audit. Only one change since the 2026-08-27 pass (#897): issue #625's perf audit was refreshed the same morning.

  • Update the perf-audit pointer in agents/DEVELOPING.md from "refreshed 2026-08-20" to 2026-08-27 and record the run's verdict (no new source-caused regressions; flagged rows are the chronic BagOfHoldingH5File per-op open cost or noise).
  • Record the still-open SQL-side hot spot the refreshed audit re-flags: Sql fsyncs once per key on save/evict because _configure_sqlite_pragmas (src/fleche/storage/sql.py:190-227) sets journal_mode=WAL but no synchronous pragma; the cheap fix (PRAGMA synchronous=NORMAL) has been flagged in every audit since 2026-05-07. Verified against the source — the function sets only foreign_keys and journal_mode.

Everything else checked and current: no merges to main since #897; open PRs (#873, #874, #887, #892, #894, #896, #797, #786, #804) and issues (#893, #895, #625) are all already recorded. eisenforschung/landau was audited in the same pass and needs no update (nothing landed since its 2026-08-25 pass; in-flight PRs #391/#394/#395/#414/#422 unchanged; spot-checked claims hold).


Generated by Claude Code

…qlFile fsync cheap fix

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012SCLy9y7aUR7F44Q7UviGo
@pmrv
pmrv merged commit 0caca71 into main Aug 28, 2026
4 checks passed
@pmrv
pmrv deleted the claude/peaceful-bell-0nndbb branch August 28, 2026 04:08
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.

2 participants