databendlabs / openraft / 18315928315

88%

Build Type

push

github

Committed by drmingdrmer

Commit Message

refactor: replace Debug with Display for RaftMsg formatting

Replace Debug formatting with Display in `RaftMsg` and implement Display
trait for `ChangeMembers` to improve log message readability. The change
removes Debug's verbose output in favor of cleaner, more concise Display
formatting.

Add three new display helper modules to support allocation-free Display
implementations:

- `DisplayBTreeMap`: formats `BTreeMap<K,V>` as "key:value,..." when
  both K and V implement Display

- `DisplayBTreeMapDebugValue`: formats `BTreeMap<K,V>` as
  "key:value,..." when K implements Display and V implements Debug

- `DisplayBTreeSet`: formats `BTreeSet<T>` as "[item,...]" when T
  implements Display

Update `RaftMsg::Initialize` to use `members.display()` instead of
`{:?}` formatting and `RaftMsg::ChangeMembership` to format `changes`
with Display instead of Debug. Remove TODO comments about avoiding Debug
formatting as the issue is now resolved.

All Display implementations iterate directly over collections without
intermediate allocations, ensuring zero-cost formatting in log paths.

Run Details

96 of 105 new or added lines in 5 files covered. (91.43%)

1 existing line in 1 file now uncovered.

11355 of 12463 relevant lines covered (91.11%)

103177.68 hits per line