databendlabs / openraft / 20391524544

88%

Build Type

push

github

Committed by drmingdrmer

Commit Message

refactor: redesign `LeaderLogIds` with explicit fields and iterator support

Replace the internal `Option<RangeInclusive<LogIdOf<C>>>` representation with
explicit `committed_leader_id`, `first`, and `last` fields. The struct now
always represents a non-empty closed range `[first, last]`, with `Option<LeaderLogIds>`
used at call sites when an empty range is needed.

Changes:
- Change `LeaderLogIds` to store `committed_leader_id`, `first`, `last` explicitly
- Add `first_ref()`, `last_ref()`, `ref_at()` methods returning `RefLogId`
- Add `first_log_id()`, `last_log_id()`, `get()`, `len()` methods
- Implement `IntoIterator` yielding `LogIdOf<C>` with `DoubleEndedIterator` support
- Update `assign_log_ids()` to return `Option<LeaderLogIds>`

Run Details

277 of 283 new or added lines in 8 files covered. (97.88%)

11 existing lines in 3 files now uncovered.

15420 of 16699 relevant lines covered (92.34%)

145236.45 hits per line