search

OpenAtomFoundation / xupercore

The kernel of xuperchain.
Apache License 2.0
61 stars 47 forks source link

consensus: optimize rollback strategy in Chained-BFT & fix bugs when … #220

Closed aucusaga closed 3 years ago

aucusaga commented 3 years ago

…reloading justify quorum certs

codecov-commenter commented 3 years ago

Codecov Report

Merging #220 (615061a) into master (b21e379) will decrease coverage by 0.60%. The diff coverage is 35.64%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #220      +/-   ##
==========================================
- Coverage   47.96%   47.35%   -0.61%     
==========================================
  Files         133      133              
  Lines       11341    11415      +74     
==========================================
- Hits         5440     5406      -34     
- Misses       4782     4881      +99     
- Partials     1119     1128       +9
Impacted Files Coverage Δ
bcs/consensus/tdpos/tdpos.go 41.76% <13.51%> (-3.46%) :arrow_down:
kernel/consensus/base/driver/chained-bft/smr.go 50.46% <40.00%> (-2.92%) :arrow_down:
bcs/consensus/xpoa/xpoa.go 56.82% <49.27%> (+4.42%) :arrow_up:
...ernel/consensus/base/driver/chained-bft/context.go 72.18% <63.15%> (+1.96%) :arrow_up:
kernel/engines/xuperos/chainmgmt.go 7.84% <0.00%> (-39.22%) :arrow_down:
kernel/engines/xuperos/engine.go 44.91% <0.00%> (-17.80%) :arrow_down:
...el/engines/xuperos/asyncworker/asyncworker_impl.go 47.48% <0.00%> (-4.32%) :arrow_down:
kernel/engines/xuperos/chain.go 48.13% <0.00%> (-1.87%) :arrow_down:
...el/consensus/base/driver/chained-bft/saftyrules.go 45.45% <0.00%> (-1.82%) :arrow_down:
... and 5 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 b21e379...615061a. Read the comment docs.