OpenAtomFoundation / xupercore

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

fix recoverUnconfirmedTx ledger check tx #259

Closed godeamon closed 3 years ago

godeamon commented 3 years ago

Description

walk 时回滚未确认交易再单独一个协程中再次执行交易,判断交易是否在账本中时,需要先判断 error 再判断是否存在账本。

codecov-commenter commented 3 years ago

Codecov Report

Merging #259 (381f2e9) into master (6293736) will increase coverage by 0.02%. The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #259      +/-   ##
==========================================
+ Coverage   46.69%   46.72%   +0.02%     
==========================================
  Files         136      136              
  Lines       12045    12048       +3     
==========================================
+ Hits         5624     5629       +5     
+ Misses       5264     5258       -6     
- Partials     1157     1161       +4     
Impacted Files Coverage Δ
bcs/ledger/xledger/state/state.go 47.17% <0.00%> (-0.18%) :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:
bcs/consensus/tdpos/tdpos.go 48.30% <0.00%> (+0.96%) :arrow_up:
bcs/consensus/tdpos/common.go 67.18% <0.00%> (+3.12%) :arrow_up:
bcs/network/p2pv1/conn.go 61.22% <0.00%> (+6.12%) :arrow_up:

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 6293736...381f2e9. Read the comment docs.