Skip to content

docs(ssi): retract overclaimed serializability guarantees - #141

Merged
gburd merged 1 commit into
masterfrom
docs/ssi-honest-claims
Sep 6, 2026
Merged

docs(ssi): retract overclaimed serializability guarantees#141
gburd merged 1 commit into
masterfrom
docs/ssi-honest-claims

Conversation

@gburd

@gburd gburd commented Sep 6, 2026

Copy link
Copy Markdown
Collaborator

Outside review (#136#140) showed the SSI docs describe the intended design, not the delivered behavior. Correcting them now, rather than after the fixes, so we are not shipping a promise we don't keep.

No engine change. The five fixes are in flight on separate branches; this is the honesty fix that shouldn't wait for them.

Related: #136, #137, #138, #139, #140

Outside review (#136-#140) showed the SSI docs describe the intended design, not
the delivered behavior. Correct them now rather than after the fixes, so we are
not shipping a promise we do not keep:

- rfc/0003: drop 'the commit-time pivot check is race-free against concurrent
  edge recording' and 'markers are reclaimed incrementally AND BOUNDED'; add a
  Known-limitations section naming #136 (write skew can commit when a
  conflicting write lands inside DB_TXN->commit; plus the separate
  different-pages-of-one-btree observation), #137/#138 (reader bookkeeping not
  fully reclaimed -> mutex-region exhaustion), #140 (DB_LOCK_SIREAD unaccounted
  in the replication commit lock-list sizing). The section is removed only when
  each item is fixed AND covered by a test.
- docs_src/api/c/txnbegin.md: add a user-facing Known-limitation note under
  DB_TXN_SNAPSHOT so applications do not rely on serializability alone yet.

No engine change; the fixes are in flight separately.
@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown

Coccinelle convention checks

No new violations. ✅

Resolved since baseline (2) -- update dist/cocci/baseline.txt to lock these in.
rule_mutex_unbalanced|MUTEX_UNBALANCED|src/crypto/mersenne/mt19937db.c|return (ret);
rule_mutex_unbalanced|MUTEX_UNBALANCED|src/mp/mp_register.c|return (ret);

@gburd
gburd merged commit 16ec17a into master Sep 6, 2026
52 of 53 checks passed
@gburd
gburd deleted the docs/ssi-honest-claims branch September 6, 2026 18:07
@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown

ABI diff vs v5.3.34 (libabigail — authoritative)

Removed exported symbols (nm -D, _NNNN version suffix normalized)

None.


Advisory: libabigail/nm is the authoritative binary-ABI check; Coccinelle is complementary source-level early warning. See dist/cocci/README.md.

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.

1 participant