You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Recheck the transactionLogAbortMarkers docs when harper#3928 merges #782
Recheck the documented default and off-values when harper#3928 merges. The docs say the setting defaults to true and that only false, 0, no or off turn it off. That matches harper#3928's head c8dfa03 (resources/transactionLogAbortMarkers.ts), not current harper main, which turns markers on only for === true. Document storage.rocks.transactionLogAbortMarkers and its 5.4 default #780 carries Depends-on: harper#3928, so it cannot merge first. But if the core PR changes its default, parsing, the 40-conflict cap or the deadline before it merges, the entry in reference/database/storage-tuning.md (and the matching sentences in reference/configuration/options.md and release-notes/v5-lincoln/5.4.md) need the same change.
The conflict deadline for internal deploy writes is not documented. The docs say a conflicting write fails with a 503 once it has been retrying longer than storage.maxTransactionQueueTime. The real deadline is the larger of that and a transaction's own timeoutBudget. Today only Harper's component-deploy recorder sets that budget (components/deploymentRecorder.ts), so client requests never see it. Decide whether it is worth a sentence. The default answer is no, because the brief said to leave out internal retry mechanics.
Close this once harper#3928 has merged and item 1 is confirmed (or fixed in a follow-up), and item 2 is decided.
#780 now also documents storage.rocks.transactionLogWriteIntents (default false), in reference/database/storage-tuning.md, reference/configuration/options.md, the three full-restart passages and the 5.4 release note. The setting is not yet on harper#3928's head (1564a87). When harper#3928 merges, also confirm that the merged code has storage.rocks.transactionLogWriteIntents with default false and the same full-restart rule.
🤖 Generated by Claude Opus 5.5; posted via @kriszyp.
Activity
kriszyp commentedon Nov 2, 2026
#780 now also documents
storage.rocks.transactionLogWriteIntents(defaultfalse), inreference/database/storage-tuning.md,reference/configuration/options.md, the three full-restart passages and the 5.4 release note. The setting is not yet on harper#3928's head (1564a87). When harper#3928 merges, also confirm that the merged code hasstorage.rocks.transactionLogWriteIntentswith defaultfalseand the same full-restart rule.🤖 Generated by Claude Opus 5.5; posted via @kriszyp.