Skip to content

[raft/memstore] Use now from context #1535

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

[raft/memstore] Use now from context #1535
the-glu wants to merge 4 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

@the-glu the-glu force-pushed the memstore_use_time_from_context branch 3 times, most recently from ece39d0 to 2b6994d Compare July 1, 2026 09:07
@the-glu the-glu force-pushed the memstore_use_time_from_context branch from 2b6994d to 0269e18 Compare July 1, 2026 09:20
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