Skip to content

Add recovery checkpoints to speed up restart after ordered shutdown#577

Merged
kjnilsson merged 3 commits intomainfrom
recovery-checkpoints
Feb 18, 2026
Merged

Add recovery checkpoints to speed up restart after ordered shutdown#577
kjnilsson merged 3 commits intomainfrom
recovery-checkpoints

Conversation

@kjnilsson
Copy link
Copy Markdown
Contributor

@kjnilsson kjnilsson commented Feb 9, 2026

Recovery checkpoints are written synchronously during terminate when min_recovery_checkpoint_interval config is set and enough log entries have been applied since the last snapshot/checkpoint. They avoid expensive log replay on restart by persisting machine state without the overhead of live index computation at write time.

@kjnilsson kjnilsson force-pushed the recovery-checkpoints branch 3 times, most recently from 797faba to e9e81ef Compare February 17, 2026 12:04
@kjnilsson kjnilsson force-pushed the recovery-checkpoints branch from e9e81ef to e28fc5d Compare February 17, 2026 13:20
Recovery checkpoints are written synchronously during terminate when
min_recovery_checkpoint_interval config is set and enough log entries
have been applied since the last snapshot/checkpoint. They avoid
expensive log replay on restart by persisting machine state without
the overhead of live index computation at write time.
@kjnilsson kjnilsson force-pushed the recovery-checkpoints branch from e28fc5d to 3b1cb37 Compare February 17, 2026 13:22
@kjnilsson kjnilsson marked this pull request as ready for review February 17, 2026 13:50
@the-mikedavis the-mikedavis self-requested a review February 17, 2026 15:46
@michaelklishin
Copy link
Copy Markdown
Contributor

I appreciate the updated snapshots, checkpoints internals doc.

Copy link
Copy Markdown
Collaborator

@the-mikedavis the-mikedavis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This makes sense to me so far. I'll give it another look tomorrow morning once I've had more ☕

I just found a tiny spacing thing so far

By avoiding second read of live indexes from disk.
@kjnilsson kjnilsson force-pushed the recovery-checkpoints branch from f819fcf to bc1f10e Compare February 18, 2026 08:21
@kjnilsson kjnilsson merged commit 93e119b into main Feb 18, 2026
7 checks passed
@dumbbell-rabbitmq dumbbell-rabbitmq deleted the recovery-checkpoints branch March 23, 2026 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants