Skip to content

[raft/memstore] Use now from context #1535

Open
the-glu wants to merge 8 commits into
interuss:masterfrom
Orbitalize:memstore_use_time_from_context
Open

[raft/memstore] Use now from context #1535
the-glu wants to merge 8 commits into
interuss:masterfrom
Orbitalize:memstore_use_time_from_context

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 use helper from timestamp to use now from context, if available, to ensure consistent time in raft context.

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

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.

3 participants