Skip to content

Commit aca1934

Browse files
committed
docs(test): clarify checkpoint-interval comment in crash truncation test
1 parent 4582462 commit aca1934

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

nodedb/tests/crash_wal_truncation.rs

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,10 @@
1515
//! Reproducing that needs three things the existing crash tests do not do, and
1616
//! all three are load-bearing:
1717
//!
18-
//! 1. A checkpoint cycle inside the test's lifetime. The default interval is
19-
//! 300s and `kv_write_survives_kill_9` finishes in ~12s, so the truncation
20-
//! window never opened and the bug sat under a green suite.
18+
//! 1. A checkpoint cycle inside the test's lifetime. The default 300s
19+
//! interval dwarfs a crash test's runtime, so the truncation window never
20+
//! opened and the bug sat under a green suite until these tests forced a
21+
//! short interval.
2122
//! 2. WAL segment ROTATION. `truncate_segments` skips the active segment
2223
//! unconditionally, so with a single segment nothing is ever deleted and
2324
//! this test would pass against the buggy code having proven nothing.

0 commit comments

Comments
 (0)