Skip to content

[raftstore] Embed memstore and use checkpoint - #1669

Merged
mickmis merged 1 commit into
interuss:masterfrom
Orbitalize:use_checkpoint
Sep 17, 2026
Merged

mickmis merged 1 commit into
interuss:masterfrom
Orbitalize:use_checkpoint

Conversation

@MariemBaccari

@MariemBaccari MariemBaccari commented Aug 28, 2026 •

Copy link
Copy Markdown
Contributor

This PR adds the use of the memstore checkpoint to ensure raft operations are atomic and embeds the memstore directly to simplify the raftstore.

Implements #1700 and #1702

@MariemBaccari
MariemBaccari marked this pull request as ready for review August 28, 2026 11:41

@mickmis mickmis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

(5fe0a16) LGTM modulo comments

Comment thread pkg/raftstore/store.go
Comment thread pkg/raftstore/store.go Outdated
@MariemBaccari
MariemBaccari force-pushed the use_checkpoint branch 2 times, most recently from 742da3a to 0a2cc46 Compare September 8, 2026 11:21
@MariemBaccari
MariemBaccari force-pushed the use_checkpoint branch 2 times, most recently from 63969e1 to cf95e9c Compare September 15, 2026 08:21
@mickmis
mickmis merged commit eea581c into interuss:master Sep 17, 2026
18 checks passed
@mickmis
mickmis deleted the use_checkpoint branch September 17, 2026 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants