databendlabs / openraft / 22949063797

88%

Build Type

push

github

Committed by drmingdrmer

Commit Message

change: in 0.10.0: `Entry<C>` to `Entry<CLID, D, NID, N>` to decouple from `RaftTypeConfig`

Refactor `Entry` to take individual type parameters (`CLID: RaftCommittedLeaderId`,
`D: AppData`, `NID: NodeId`, `N: Node`) instead of a single `C: RaftTypeConfig`,
continuing the type-decoupling effort to break circular type dependencies.

The `declare_raft_types!` macro default for `Entry` now expands to the
fully-qualified form. A `DefaultEntryOf<C>` alias is added for the common
concrete instantiation.

Changes:
- Add `DefaultEntryOf<C>` type alias for `Entry<CommittedLeaderIdOf<C>, DOf<C>, NodeIdOf<C>, NodeOf<C>>`
- Update `declare_raft_types!` default Entry to expanded generic form
- Replace `Entry<TypeConfig>` with `EntryOf<TypeConfig>` in memstore crates

Upgrade tip:

If you use `Entry<C>` directly, switch to one of:
- `EntryOf<C>` (alias for `C::Entry`, the associated type)
- `DefaultEntryOf<C>` (alias for the concrete `Entry` struct with types from C)

Before:
    use openraft::Entry;
    type MyEntry = Entry<MyConfig>;

After:
    use openraft::alias::EntryOf;       // for C::Entry
    use openraft::alias::DefaultEntryOf; // for concrete Entry struct
    type MyEntry = EntryOf<MyConfig>;

If you construct entries with `blank_ent()` or `membership_ent()`,
add turbofish: `blank_ent::<MyConfig>(...)`.

Run Details

58 of 60 new or added lines in 10 files covered. (96.67%)

5 existing lines in 2 files now uncovered.

15932 of 18190 relevant lines covered (87.59%)

145874.74 hits per line