Skip to content

[Raft] Add generic raftstore#1525

Open
MariemBaccari wants to merge 2 commits into
interuss:masterfrom
Orbitalize:add_generic_raftstore
Open

[Raft] Add generic raftstore#1525
MariemBaccari wants to merge 2 commits into
interuss:masterfrom
Orbitalize:add_generic_raftstore

Conversation

@MariemBaccari

@MariemBaccari MariemBaccari commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Implements the generic raftstore which processes and applies committed entries.

@MariemBaccari MariemBaccari marked this pull request as ready for review June 15, 2026 08:14
@MariemBaccari MariemBaccari marked this pull request as draft June 15, 2026 08:17
@MariemBaccari MariemBaccari force-pushed the add_generic_raftstore branch from 2fd1b75 to 4e0c128 Compare June 15, 2026 08:23
@MariemBaccari MariemBaccari marked this pull request as ready for review June 15, 2026 08:24
@MariemBaccari MariemBaccari force-pushed the add_generic_raftstore branch 5 times, most recently from 7170586 to 1e2ae99 Compare June 16, 2026 08:49
@barroco barroco added the dss-raft Relating to the application-layer consensus implemenation based on raft label Jun 17, 2026
@MariemBaccari MariemBaccari force-pushed the add_generic_raftstore branch 2 times, most recently from 7746510 to ff8895f Compare June 18, 2026 07:38
@MariemBaccari MariemBaccari force-pushed the add_generic_raftstore branch from ff8895f to 22043b4 Compare June 18, 2026 08:01
@MariemBaccari MariemBaccari force-pushed the add_generic_raftstore branch 4 times, most recently from 7f0c1ad to 14951fc Compare June 19, 2026 13:09
@MariemBaccari MariemBaccari force-pushed the add_generic_raftstore branch from 14951fc to a52ee35 Compare June 22, 2026 09:13

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

Partial review 1: Please create a separate PR with all changes which touche the current sqlstore implementation.

Comment thread pkg/timestamp/timestamp.go Outdated
Comment thread pkg/timestamp/timestamp.go Outdated

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

Partial review 2

Comment thread pkg/timestamp/timestamp.go Outdated
Comment thread pkg/timestamp/timestamp.go Outdated
@MariemBaccari MariemBaccari force-pushed the add_generic_raftstore branch 14 times, most recently from c2c055f to 27f5004 Compare June 29, 2026 12:25
@MariemBaccari MariemBaccari marked this pull request as draft June 29, 2026 13:12
@MariemBaccari MariemBaccari force-pushed the add_generic_raftstore branch 4 times, most recently from 269f2fe to f25dff4 Compare June 30, 2026 13:50
@MariemBaccari MariemBaccari force-pushed the add_generic_raftstore branch 3 times, most recently from 1e6976c to 45bc00f Compare July 1, 2026 09:29
@MariemBaccari MariemBaccari force-pushed the add_generic_raftstore branch from 45bc00f to 05a69b1 Compare July 1, 2026 09:40
@MariemBaccari MariemBaccari marked this pull request as ready for review July 1, 2026 09:40
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

None yet

Development

Successfully merging this pull request may close these issues.

2 participants