databendlabs / openraft / 18844266382

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%)

2 existing lines in 1 file now uncovered.

12213 of 13368 relevant lines covered (91.36%)

138412.73 hits per line