OpenAtomFoundation / xupercore

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

Fix pack block tx order for write and readonly tx #304

Closed godeamon closed 1 year ago

godeamon commented 2 years ago

遍历交易时,判断有没有相同key的只读或者读写交易,需要保证只读交易在读写交易前打包。

gaoweibupt commented 1 year ago

lgtm

codecov-commenter commented 1 year ago

Codecov Report

Merging #304 (62636c8) into master (26f7097) will increase coverage by 0.93%. The diff coverage is 56.83%.

@@            Coverage Diff             @@
##           master     #304      +/-   ##
==========================================
+ Coverage   45.30%   46.24%   +0.93%     
==========================================
  Files         142      142              
  Lines       12488    12721     +233     
==========================================
+ Hits         5658     5883     +225     
+ Misses       5669     5660       -9     
- Partials     1161     1178      +17     
Impacted Files Coverage Δ
bcs/ledger/xledger/tx/mempool.go 42.47% <46.51%> (+11.99%) :arrow_up:
bcs/ledger/xledger/tx/tx.go 39.87% <50.00%> (+<0.01%) :arrow_up:
bcs/ledger/xledger/state/state.go 46.78% <59.09%> (+0.10%) :arrow_up:
bcs/ledger/xledger/tx/node.go 71.67% <82.55%> (+31.85%) :arrow_up:
bcs/consensus/xpoa/xpoa.go 51.26% <0.00%> (-1.02%) :arrow_down:
kernel/consensus/base/driver/chained-bft/smr.go 41.33% <0.00%> (+0.74%) :arrow_up:
bcs/contract/xvm/code_manager.go 66.36% <0.00%> (+2.72%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.