Skip to content

[raft/memstore] Add Checkpoint/Restore - #1539

Merged
mickmis merged 1 commit into
interuss:masterfrom
Orbitalize:memstore_rid_clone
Jul 30, 2026
Merged

[raft/memstore] Add Checkpoint/Restore#1539
mickmis merged 1 commit into
interuss:masterfrom
Orbitalize:memstore_rid_clone

Conversation

@the-glu

@the-glu the-glu commented Jun 16, 2026

Copy link
Copy Markdown
Member

This PR is part of a new chain, implementing memstore.

#1525 (Generic raftstore) -> #1527 (Base memstore) -> #1529 (Aux memstore) -> #1530 (Snapshots) -> #1534 (Rid memstore) -> #1535 (Correct use of now) -> #1539 (Checkpoint) -> #1542 (Scd memstore) -> #1528 (First PR with raft using memstore)


It adds a Checkpoint/Restore to every memstore, to be used by raft in Transact() operations, allowing it to quickly restore the memstore back to the start of a transaction should an error occur, without the need for complex revert operations.

@barroco barroco added the dss-raft Relating to the application-layer consensus implemenation based on raft label Jun 17, 2026
@the-glu
the-glu force-pushed the memstore_rid_clone branch from 97c5178 to 73de5df Compare June 18, 2026 08:30
@the-glu
the-glu force-pushed the memstore_rid_clone branch from 73de5df to 9f869ba Compare June 19, 2026 13:28
@the-glu
the-glu force-pushed the memstore_rid_clone branch 2 times, most recently from 03b7690 to 6a77da6 Compare June 29, 2026 07:04
@the-glu
the-glu force-pushed the memstore_rid_clone branch 4 times, most recently from d6be655 to 8a60b20 Compare July 1, 2026 09:23
@the-glu
the-glu force-pushed the memstore_rid_clone branch from 8a60b20 to 9a4f64d Compare July 13, 2026 08:58
Comment thread pkg/memstore/store.go Outdated
Comment thread pkg/rid/store/memstore/store.go
Comment thread pkg/scd/store/memstore/store.go Outdated
Comment thread pkg/aux_/store/memstore/store.go
@the-glu
the-glu force-pushed the memstore_rid_clone branch 5 times, most recently from ba3f611 to 9ef98af Compare July 30, 2026 14:57
@the-glu
the-glu requested a review from mickmis July 30, 2026 14:59

@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.

LGTM modulo CI

Comment thread pkg/rid/store/memstore/store.go Outdated
@the-glu
the-glu force-pushed the memstore_rid_clone branch from 9ef98af to 76897c3 Compare July 30, 2026 15:49
@the-glu
the-glu requested a review from mickmis July 30, 2026 15:49
@mickmis
mickmis merged commit 48b9877 into interuss:master Jul 30, 2026
12 checks passed
@mickmis
mickmis deleted the memstore_rid_clone branch July 30, 2026 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dss-raft Relating to the application-layer consensus implemenation based on raft

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants