Repository navigation
docs: reconcile the PRD and the tracker after the data-hygiene wave - #572
Merged
Merged
Conversation
The parent-owned fields, and only those: the PRD version line, the PRD codebase version, the PRD footer stamp, the tracker's Last-synced line, the tracker's Summary table, the tracker's In Progress table — plus one measurement correction inside the wave's own section. - PRD version 4.17-scheduled-reclaim -> 4.19-data-hygiene - PRD codebase version 0.35.0 -> 0.35.2 (cmd/leankg/VERSION is 0.35.2 since the release merge); this is the drift the previous reconciliation left behind, one release later - the footer stamp and the Last-synced line now describe THIS wave and name its seven PRs - the Summary counts and the In Progress table: the nine-defect wave's row is closed (all nine merged) and the data-hygiene wave's row is added - correction: the FTS measurement in the v4.19-scheduled-reclaim-real section was wrong. The index was 73,728 B, not 9.4 MB, and there is no "3,065 segments" figure in the store; the FTS tables hold 43,370,176 B (31 % of the store) and the segment structures inside them hold 16,728,064 B, of which one compaction reclaims 6,152,192 B with every keyword row intact (11,584 -> 11,584). The correction is also posted on PR #566.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The parent-owned fields, and only those — the PRD version line, the PRD codebase version, the PRD footer stamp, the tracker's
**Last synced:**line, the tracker Summary table, the tracker In Progress table — plus one measurement correction inside the wave's own section.**Version:**4.17-scheduled-reclaim → 4.19-data-hygiene**Codebase Version:**0.35.0 → 0.35.2 (cmd/leankg/VERSIONis 0.35.2 since the release merge) — this is the same drift the previous reconciliation left behind, one release later**Last synced:**line now describe this wave and name its seven PRs (memory: content identity, bank compaction, fleet memory sweep, MCP bank delete #564, fix(index): a markdown heading stores its own text, not its descendants' #565, maintain: make the hourly pass do its job, and say what it did #566, store: give a new store a page that can hold three vectors #567, portfolioreg: an automatic registry prune, and a store whose index root moved is reported #568, fix(telemetry): sweep the ledger whatever the capture level, and let it shrink #570, fix(index): reconcile the store with HEAD, and report head_sync #571)Measurement correction. The FTS figures in the v4.19-scheduled-reclaim-real section were wrong, and are corrected here:
dbstaton a copy of the live store (with the-wal) sayselements_fts_idxis 73,728 B (18 pages) — not 9.4 MB — and there is no "3,065 segments" figure in the store. The "31 %" belongs to the FTS tables as a whole (43,370,176 B of 138,678,272 B). What one compaction actually reclaims, measured by running the mergedCompactFTSagainst a copy of the live store:elements_fts_dataelements_fts_idx6,152,192 B (6.15 MB) of merged segment space reclaimed, with every keyword row intact. The same correction is posted on PR #566.
The fix and its tests are unaffected; only the measured attribution was wrong.