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.

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

introduce follower replication #11455

Closed
wants to merge 1 commit into from

Conversation

Fullstop000
Copy link
Contributor

Signed-off-by: Fullstop000 fullstop1005@gmail.com

Implement follower replication

Relate to #11357

@codecov-io
Copy link

codecov-io commented Dec 16, 2019

Codecov Report

Merging #11455 into master will increase coverage by 0.38%.
The diff coverage is 79.29%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #11455      +/-   ##
==========================================
+ Coverage   66.08%   66.47%   +0.38%     
==========================================
  Files         401      403       +2     
  Lines       36627    36839     +212     
==========================================
+ Hits        24205    24488     +283     
+ Misses      10943    10837     -106     
- Partials     1479     1514      +35
Impacted Files Coverage Δ
raft/raft.go 87.07% <71.07%> (-3.84%) ⬇️
raft/group.go 88.67% <88.67%> (ø)
clientv3/ordering/kv.go 16.17% <0%> (-69.12%) ⬇️
etcdserver/api/v3rpc/util.go 51.61% <0%> (-16.13%) ⬇️
clientv3/namespace/watch.go 87.87% <0%> (-6.07%) ⬇️
raft/tracker/inflights.go 91.83% <0%> (-4.09%) ⬇️
pkg/testutil/recorder.go 77.77% <0%> (-3.71%) ⬇️
etcdserver/api/v3compactor/revision.go 79.31% <0%> (-3.45%) ⬇️
pkg/adt/interval_tree.go 84.21% <0%> (-3.01%) ⬇️
proxy/grpcproxy/watch.go 89.94% <0%> (-2.96%) ⬇️
... and 19 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 52fba43...25dfdbc. Read the comment docs.

@Fullstop000 Fullstop000 marked this pull request as ready for review December 31, 2019 05:27
@Fullstop000 Fullstop000 force-pushed the follower-replication branch 2 times, most recently from 1945b4f to c4e5612 Compare February 27, 2020 07:21
Signed-off-by: Fullstop000 <fullstop1005@gmail.com>
@hicqu
Copy link
Contributor

hicqu commented Mar 10, 2020

@xiang90 Could we hold a meeting to discuss this? So we can speed up it.

@Fullstop000
Copy link
Contributor Author

@xiang90 we've test this feature in TiKV and it results reducing some network transport cost. So we believe that this feature can facilitate cross-IDC transmission :)

@stale
Copy link

stale bot commented Jun 18, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed after 21 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jun 18, 2020
@stale stale bot closed this Jul 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

3 participants