OpenAtomFoundation / xupercore

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

fix confirm tx and p2p map cocurrent read #245

Closed godeamon closed 3 years ago

godeamon commented 3 years ago

Description

修改 confirem tx 耗时太长,以及删除 p2p 中的多余代码。

Fixes # (issue)

Type of change

codecov-commenter commented 3 years ago

Codecov Report

Merging #245 (d50f679) into master (64bb9f8) will decrease coverage by 0.15%. The diff coverage is 45.16%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #245      +/-   ##
==========================================
- Coverage   46.83%   46.68%   -0.16%     
==========================================
  Files         136      136              
  Lines       12006    12028      +22     
==========================================
- Hits         5623     5615       -8     
- Misses       5224     5256      +32     
+ Partials     1159     1157       -2     
Impacted Files Coverage Δ
kernel/network/p2p/dispatcher.go 89.61% <ø> (+3.53%) :arrow_up:
bcs/ledger/xledger/tx/mempool.go 31.20% <30.43%> (-0.96%) :arrow_down:
bcs/ledger/xledger/state/state.go 47.66% <83.33%> (+0.13%) :arrow_up:
bcs/ledger/xledger/tx/tx.go 39.35% <100.00%> (ø)
bcs/network/p2pv1/conn.go 53.19% <0.00%> (-6.39%) :arrow_down:
bcs/consensus/tdpos/common.go 64.06% <0.00%> (-3.13%) :arrow_down:
bcs/consensus/tdpos/tdpos.go 47.34% <0.00%> (-1.94%) :arrow_down:
bcs/ledger/xledger/tx/node.go 40.90% <0.00%> (-1.82%) :arrow_down:
bcs/network/p2pv2/server.go 51.33% <0.00%> (-1.07%) :arrow_down:
... and 2 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 64bb9f8...d50f679. Read the comment docs.