databendlabs / openraft / 22906186010

88%

Build Type

push

github

Committed by drmingdrmer

Commit Message

change: in 0.10.0: decouple `StoredMembership`, `SnapshotSignature`, `SnapshotMeta`, `Snapshot` from `RaftTypeConfig`

Replace `C: RaftTypeConfig` with the specific type parameters each
struct actually needs, continuing the type decoupling effort.

- `StoredMembership<C>` → `StoredMembership<CLID, NID, N>`
  Uses `LogId<CLID>` and `Membership<NID, N>`.

- `SnapshotSignature<C>` → `SnapshotSignature<CLID>`
  Only uses `LogId<CLID>`.

- `SnapshotMeta<C>` → `SnapshotMeta<CLID, NID, N>`
  Uses `LogId<CLID>` and `StoredMembership<CLID, NID, N>`.

- `Snapshot<C>` → `Snapshot<CLID, NID, N, SD>`
  Uses `SnapshotMeta<CLID, NID, N>` and snapshot data `SD`.

Type aliases `StoredMembershipOf<C>`, `SnapshotSignatureOf<C>`,
`SnapshotMetaOf<C>`, and `SnapshotOf<C>` are added for callers
that have `C: RaftTypeConfig` available.

Upgrade tip:

    // Before
    StoredMembership<MyConfig>
    SnapshotMeta<MyConfig>
    Snapshot<MyConfig>

    // After (with type alias)
    use openraft::alias::StoredMembershipOf;
    StoredMembershipOf<MyConfig>
    SnapshotMetaOf<MyConfig>
    SnapshotOf<MyConfig>

    // After (direct)
    StoredMembership<MyClid, MyNodeId, MyNode>
    SnapshotMeta<MyClid, MyNodeId, MyNode>
    Snapshot<MyClid, MyNodeId, MyNode, MySnapshotData>

Run Details

112 of 120 new or added lines in 29 files covered. (93.33%)

5 existing lines in 2 files now uncovered.

15898 of 18158 relevant lines covered (87.55%)

146199.09 hits per line