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

Local reader #17

Merged
merged 13 commits into from
May 31, 2021
Merged

Local reader #17

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
cfd6d3a
Local reader
overvenus Nov 26, 2018
e489569
clarify ambiguous
overvenus Nov 28, 2018
3deaab9
Address comments
overvenus Nov 29, 2018
4f27818
Merge branch 'master' into localreader
overvenus Dec 24, 2018
73df9e9
Merge branch 'master' into localreader
overvenus Dec 29, 2018
b77accc
Address lints
overvenus Dec 29, 2018
eed0428
Merge branch 'localreader' of github.com:tikv/rfcs into localreader
overvenus Dec 29, 2018
31dc7ad
Merge branch 'master' into localreader
Hoverbear Feb 20, 2019
efa4ae6
Address comments
overvenus Feb 26, 2019
1b127a7
Merge branch 'localreader' of github.com:tikv/rfcs into localreader
overvenus Feb 26, 2019
c717ef8
Address comments
overvenus Feb 28, 2019
5352ab6
Merge branch 'master' into localreader
overvenus May 22, 2019
d764f76
Merge branch 'master' into localreader
zhangjinpeng87 May 31, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Address comments 
Signed-off-by: Neil Shen <[email protected]>
  • Loading branch information
@overvenus
overvenus committed Nov 29, 2018
commit 3deaab96bdb55763abd60cfe0f7432e84c0973c9
10 changes: 6 additions & 4 deletions text/2018-01-03-local-reader.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,9 @@ linearizability.
## Local reader

The local reader uses ReadDelegates (delegate) to handle requests. Every
Hoverbear marked this conversation as resolved.
Show resolved Hide resolved
delegate is owned by a Raft peer which belongs to the raftstore, they
communicate via a channel, and each pair of them shares an atomic `LeaderLease`.
delegate is owned by a Raft peer which belongs to the raftstore. The delegate
and the peer communicate via a channel, and each pair of them shares an atomic
`LeaderLease`.

A peer can do local read as long as it holds the following conditions (only list
Hoverbear marked this conversation as resolved.
Show resolved Hide resolved
the most important):
Expand Down Expand Up @@ -84,8 +85,9 @@ them can be resolved by expiring atomic `LeaderLease`.
Local reader is blocked before taking the snapshot while the leadership has
changed.

It is addressed by expiring the leader lease so the request failed lease check
after reading.
It is addressed by expiring the leader lease. After reading, it will be checked
whether the lease is outdated. If yes, the reading results will not be returned
back.

### Case 2

Expand Down