Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Linearizable read with ReadIndex #965

Closed
Tracked by #806
drmingdrmer opened this issue Dec 8, 2023 · 1 comment · Fixed by #964
Closed
Tracked by #806

Linearizable read with ReadIndex #965

drmingdrmer opened this issue Dec 8, 2023 · 1 comment · Fixed by #964

Comments

@drmingdrmer
Copy link
Member

drmingdrmer commented Dec 8, 2023 •
edited
Loading

Obtains a ReadIndex from RaftCore if it remains the leader, and blocks until the state machine applies up to the ReadIndex. This process ensures that the application observes all state visible to a preceding read operation.
@drmingdrmer drmingdrmer mentioned this issue Dec 8, 2023
Open
4 tasks
@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 8, 2023

👋 Thanks for opening this issue!

Get help or engage by:

  • /help : to print help messages.
  • /assignme : to assign this issue to you.

drmingdrmer added a commit to drmingdrmer/openraft that referenced this issue Dec 8, 2023
@drmingdrmer
Feature: add Raft::ensure_linearizable() to ensure linearizable read
79372b4 
The `Raft::is_leader()` method does not fully ensure linearizable read
operations and is deprecated in this version. Instead, applications
should use the `Raft::ensure_linearizable()` method to guarantee
linearizability.

Under the hood, `Raft::ensure_linearizable()` obtains a `ReadIndex` from
`RaftCore` if it remains the leader, and blocks until the state
machine applies up to the `ReadIndex`. This process ensures that the
application observes all state visible to a preceding read operation.

- Fix: databendlabs#965

Upgrade tip:

Replace `Raft::is_leader()` with `Raft::ensure_linearizable()`.
@drmingdrmer drmingdrmer mentioned this issue Dec 8, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Feature: add Raft::ensure_linearizable() to ensure linearizable read drmingdrmer/openraft
1 participant
@drmingdrmer