OpenAtomFoundation / xupercore

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

Fix mempool delete confirm tx #247

Closed godeamon closed 3 years ago

godeamon commented 3 years ago

Description

回滚区块时,mempool 删除交易需要从 confirmed 列表中也删除。 Fixes # (issue)

Type of change

codecov-commenter commented 3 years ago

Codecov Report

Merging #247 (a4c4ba0) into master (f3461a4) will decrease coverage by 0.11%. The diff coverage is 33.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #247      +/-   ##
==========================================
- Coverage   46.74%   46.63%   -0.12%     
==========================================
  Files         136      136              
  Lines       12028    12036       +8     
==========================================
- Hits         5623     5613      -10     
- Misses       5248     5263      +15     
- Partials     1157     1160       +3     
Impacted Files Coverage Δ
bcs/ledger/xledger/state/state.go 47.41% <0.00%> (-0.26%) :arrow_down:
bcs/ledger/xledger/tx/mempool.go 31.35% <33.33%> (+0.15%) :arrow_up:
bcs/ledger/xledger/tx/tx.go 39.10% <50.00%> (-0.26%) :arrow_down:
bcs/consensus/xpoa/xpoa.go 53.76% <0.00%> (-2.16%) :arrow_down:
bcs/consensus/tdpos/tdpos.go 48.30% <0.00%> (-0.97%) :arrow_down:
bcs/consensus/tdpos/schedule.go 47.70% <0.00%> (-0.92%) :arrow_down:
kernel/consensus/base/driver/chained-bft/smr.go 41.87% <0.00%> (-0.83%) :arrow_down:

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 f3461a4...a4c4ba0. Read the comment docs.