databendlabs / openraft / 23351530562

86%

Build Type

push

github

Committed by drmingdrmer

Commit Message

feat: add `committed` field to `RaftMetrics` and `RaftDataMetrics`

`RaftMetrics` exposed log state fields (`last_log_index`, `last_applied`,
`snapshot`, `purged`) but was missing `committed` — the last log ID this
node knows to be safe for local application. This is the **local
committed** value, which may lag behind the cluster-committed frontier
due to network delays or out-of-order RPC delivery. Internally,
`committed` was already tracked via `RaftState::committed()`; it just
was not surfaced in metrics.

Changes:
- Add `committed: Option<LogIdOf<C>>` to `RaftMetrics` and `RaftDataMetrics`
- Add `Metric::Committed` variant with `PartialEq`/`PartialOrd` support
- Add `set_committed_index()` to `MetricsRecorder` trait and `forward_metrics()`
- Add `Wait::committed_index()` and `Wait::committed_index_at_least()`
- Add `openraft.log.index.committed` gauge in OpenTelemetry integration
- Add unit tests for all new public items

Coverage Stats

69 of 71 new or added lines in 7 files covered. (97.18%)

9 existing lines in 5 files now uncovered.

16004 of 18577 relevant lines covered (86.15%)

149000.58 hits per line