search

bosnet / sebak

SEBAK, the next BOScoin Blockchain
https://devteam.blockchainos.org
GNU General Public License v3.0
44 stars 15 forks source link

Don't store the whole transaction in BlockOperation #907

Closed Geod24 closed 5 years ago

Geod24 commented 5 years ago

Since we only need the sequenceID, reduce the memory footprint.

codecov-io commented 5 years ago

Codecov Report

Merging #907 into master will increase coverage by 0.06%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #907      +/-   ##
==========================================
+ Coverage   60.28%   60.35%   +0.06%     
==========================================
  Files         153      153              
  Lines       10838    10838              
==========================================
+ Hits         6534     6541       +7     
+ Misses       3542     3534       -8     
- Partials      762      763       +1
Flag Coverage Δ
#integration_tests_long_term 44.53% <100%> (+0.1%) :arrow_up:
#integration_tests_node 40.29% <100%> (-0.06%) :arrow_down:
#unittests 48.46% <100%> (ø) :arrow_up:
Impacted Files Coverage Δ
lib/block/operation.go 58.67% <100%> (ø) :arrow_up:
lib/network/validator_connection_manager.go 86.95% <0%> (+1.97%) :arrow_up:
lib/common/message.go 42.1% <0%> (+10.52%) :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 e9f09b1...6ac6966. Read the comment docs.