databendlabs / openraft / 20677454702

88%

Build Type

push

github

Committed by drmingdrmer

Commit Message

change: implement truncate_after for RaftLogStorage

Add new `truncate_after(last_log_id: Option<LogId>)` method to provide clearer
log truncation semantics. This is a new storage API that implementers must provide.

Changes:
- Add `truncate_after()` method to RaftLogStorage trait
- Implement `truncate_after()` to delegate to deprecated `truncate()` by default
- Update internal raft_core to use `truncate_after()` instead of `truncate()`
- Add tests for truncating after specific index and with None (delete all)

Upgrade tip:

Storage implementers need to add the new `truncate_after()` method:

impl RaftLogStorage<C> for MyStorage {
    async fn truncate_after(&mut self, last_log_id: Option<LogIdOf<C>>) -> Result<(), io::Error> {
        // Your implementation here
    }
}

The default implementation delegates to the deprecated `truncate()` method,
so existing implementations continue to work but should migrate to the new API.

Run Details

80 of 80 new or added lines in 8 files covered. (100.0%)

3 existing lines in 1 file now uncovered.

15594 of 17809 relevant lines covered (87.56%)

159998.56 hits per line