Closed ZuLiangWang closed 1 year ago
Merging #205 (604044a) into main (7f1354e) will increase coverage by
20.49%
. The diff coverage is57.23%
.
@@ Coverage Diff @@
## main #205 +/- ##
===========================================
+ Coverage 39.66% 60.15% +20.49%
===========================================
Files 34 42 +8
Lines 4213 5075 +862
===========================================
+ Hits 1671 3053 +1382
+ Misses 2317 1632 -685
- Partials 225 390 +165
Impacted Files | Coverage Δ | |
---|---|---|
pkg/coderr/code.go | 0.00% <ø> (ø) |
|
server/cluster/metadata/types.go | 0.00% <0.00%> (ø) |
|
server/coordinator/procedure/util.go | 0.00% <ø> (ø) |
|
server/storage/meta.go | 0.00% <ø> (ø) |
|
server/storage/storage_impl.go | 57.21% <0.00%> (-5.10%) |
:arrow_down: |
server/member/watch_leader.go | 40.62% <20.00%> (-2.71%) |
:arrow_down: |
server/cluster/manager.go | 38.96% <23.52%> (ø) |
|
server/coordinator/factory.go | 76.68% <27.27%> (+72.22%) |
:arrow_up: |
.../operation/transferleader/batch_transfer_leader.go | 33.80% <33.80%> (ø) |
|
server/member/member.go | 47.39% <45.00%> (-1.32%) |
:arrow_down: |
... and 19 more |
Which issue does this PR close?
Closes #173
Rationale for this change
As it mentioned in issue, we need to support open shard concurrently to improve failover performence.
What changes are included in this PR?
batch_transfer_leader
, it is used to open multiple shards concurrently in batches.Are there any user-facing changes?
None.
How does this change test
Pass all unit tests and integration tests.