hyperledger / fabric-x-committer / 30988812892

91%

Build Type

push

github

Committed by web-flow

Commit Message

[vc] Add background snapshot hash worker (#714)

[vc] Add background snapshot hash worker

#### Type of change

- New feature
- Test update

#### Description

- Add a single-goroutine background worker (`runSnapshotHashWorker`)
that
  the committer enqueues immediately after a `_snapshot` txID commits
  (before status is reported downstream), driving the `_snapshot` record
  PENDING -> IN_PROGRESS -> COMPLETED.
- Hash algorithm: per-table SHA-256 over rows in primary-key order with
  length-prefixed encoding `len(key)||key||len(value)||value`, combined
  across tables in sorted table-name order. The hashed table set is
  derived from the `ns__meta` namespace registry plus the fixed system
  tables `ns__meta`, `ns__config`, and `tx_status`; `ns__snapshot`,
  `ns__checkpoint`, and `metadata` are excluded as bookkeeping.
  Per-table hashing runs in parallel (bounded by
  `max-workers-for-snapshot-hash`) with keyset-paginated reads
  (`snapshot-hash-batch-size`) against a short-lived pool opened on the
  clone database.
- Status transitions are retried, versioned UPDATEs against the
  `_snapshot` row (`updateSnapshotState`), reusing the
  `decodeSnapshotState`/`encodeSnapshotState` helpers from the
  clone-creation path (#701). Only one snapshot is hashed at a time
  (single serial worker).
- New `ResourceLimitsConfig` knobs: `max-workers-for-snapshot-hash`
  (default 4) and `snapshot-hash-batch-size` (default 1000), documented
  in the sample config and performance-tuning guide.
- Adds `utils/statedb.TxStatusTableName`.

#### Additional details (Optional)

Tests (`service/vc/snapshot_hash_test.go`): deterministic hash across
namespaces and `tx_status`, hash reflects state changes and excludes the
`_snapshot`/`_checkpoint` bookkeeping namespaces, enqueue/decode
guard behavior, end-to-end commit -> COMPLETED via the background
worker, idempotent re-enqueue, and enqueue cancellation handling. All
async assertions use `require.Eventually`, no `time.Sleep`.

... (continued)

Coverage Stats

202 of 254 new or added lines in 5 files covered. (79.53%)

26 existing lines in 8 files now uncovered.

10770 of 11867 relevant lines covered (90.76%)

33229.3 hits per line