databendlabs / openraft / 18844266369

88%

main: 88%

Build Type

push

github

Committed by drmingdrmer

Commit Message

change: in 0.10.0: remove ResponderBuilder, use generic `Responder<T>`

Replace the ResponderBuilder abstraction with direct generic `Responder<T>`.
This simplifies the type system by eliminating an unnecessary builder layer.

There is no need to let OpenRaft build a `Responder` from the
`AppData`(`RaftTypeConfig::D`), but instead, letting the user build it and
pass it to OpenRaft to use is much simpler.

Changes:
- Remove `ResponderBuilder` trait
- Change `RaftTypeConfig::ResponderBuilder` to `Responder<T>`
- Make `OneshotResponder<C, T>` generic over response type
- Update `Raft::client_write_ff()` to accept a new responder parameter

Other changes:
- Disable test for type_param, only run it locally

Upgrade tips:

Build a `Responder` and pass it to `Raft::client_write_ff()`.

Run Details

27 of 27 new or added lines in 5 files covered. (100.0%)

3 existing lines in 2 files now uncovered.

12212 of 13368 relevant lines covered (91.35%)

138342.2 hits per line