hyperledger / fabric-x-committer / 29748511094

92%

main: 91%

Build Type

Pull #685

github

Committed by cendhu

Commit Message

[vc, coordinator, verifier] treat _snapshot and _checkpoint as system namespaces

Add committerpb.SnapshotNamespaceID and CheckpointNamespaceID to the
systemNamespaces list so their ns_<id> tables are created via the
existing namespace bootstrap path, alongside _meta and _config. No new
SQL DDL and no timestamp columns.

Recognize _snapshot and _checkpoint as first-class system namespaces
across policy validation, the dependency graph, and the VC preparer,
mirroring existing _meta/_config handling.

- Add utils.IsSystemNamespace covering _meta, _config, _snapshot, and
  _checkpoint, and use it in policy validation and dependency-graph
  read-key construction to avoid duplicating the 4-way guard.
- policy: reject _snapshot/_checkpoint as ordinary application namespace
  policies.
- dependencygraph: skip the normal _meta namespace-version read
  dependency for system namespaces.
- vc preparer: skip the _meta dependency for both; forward the _snapshot
  marker TX to the committer as a single new write into the _snapshot
  namespace (key = tx_id, value = SnapshotState{TxRef}); retain the
  single ReadWrite for _checkpoint. Fail the preparer worker rather than
  silently dropping a _snapshot write, since that would fork state across
  deterministic replicas.

Add table-driven tests for each package.

Signed-off-by: Senthilnathan <cendhu@gmail.com>

Pull Request Pull Request #685: [vc, coordinator, verifier] treat _snapshot and _checkpoint as system namespaces

Coverage Stats

38 of 40 new or added lines in 4 files covered. (95.0%)

14 existing lines in 5 files now uncovered.

10500 of 11446 relevant lines covered (91.74%)

42954.88 hits per line