databendlabs / openraft / 20467363442

88%

Build Type

push

github

Committed by drmingdrmer

Commit Message

fix: handle empty `limited_get_log_entries` gracefully instead of panicking

When `limited_get_log_entries()` returns an empty vector for a non-empty range,
which violates its API contract, the replication now treats it as a heartbeat
instead of panicking. A 500ms sleep is added to avoid a tight loop since the
log_id_range won't advance when no logs are returned.

Changes:
- Handle empty result in `read_log_entries()` by treating it as a heartbeat
- Add test infrastructure in memstore to simulate empty returns
- Add test `empty_limited_get_log_entries` to verify the fix

- Fix: #1601

Run Details

19 of 19 new or added lines in 2 files covered. (100.0%)

5 existing lines in 2 files now uncovered.

15818 of 17190 relevant lines covered (92.02%)

163151.63 hits per line