databendlabs / openraft / 27152208211

87%

Build Type

push

github

Committed by drmingdrmer

Commit Message

refactor: quorum: remove AsJoint trait

# Summary

`AsJoint` was pure sugar for viewing a `Vec<QS>` as a borrowed joint quorum
set: `vec.as_joint()` desugared to `Joint::new(&vec[..])`. It is removed. The
production callers that used it only to enumerate a joint config's voter ids
now compute that id-union directly, and the remaining callers build `Joint`
directly. The trait is `pub(crate)`, so there is no public API or behavior
change.

# Details

- In `Membership`, the `as_joint().ids()` calls backing voter-id enumeration
  now collect the deduplicated id-union straight from the configs. This drops
  a throwaway `Joint` allocation per call and leaves the `QuorumSet` import
  there unused, so it is removed alongside `AsJoint`.
- `Joint` and its quorum-set behavior are untouched; the trait only added a
  `Vec`-receiver shortcut, so the test and bench call sites now construct the
  same `Joint` via `Joint::new`.

Coverage Stats

21 of 21 new or added lines in 3 files covered. (100.0%)

3 existing lines in 2 files now uncovered.

16483 of 19025 relevant lines covered (86.64%)

144747.08 hits per line