crypto-org-chain / chain-main

Cronos POS Chain⛓: Croeseid Testnet and beyond development
https://cronos-pos.org/
Other
497 stars 352 forks source link

Problem: memiavl result is different when replaying write-ahead-log #995

Closed yihuang closed 1 year ago

yihuang commented 1 year ago

Solution:

👮🏻👮🏻👮🏻 !!!! REFERENCE THE PROBLEM YOUR ARE SOLVING IN THE PR TITLE AND DESCRIBE YOUR SOLUTION HERE !!!! DO NOT FORGET !!!! 👮🏻👮🏻👮🏻

PR Checklist:

Thank you for your code, it's appreciated! :)

codecov[bot] commented 1 year ago

Codecov Report

Merging #995 (61f02a0) into master (8dbdee7) will decrease coverage by 0.02%. The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master     #995      +/-   ##
==========================================
- Coverage   33.06%   33.04%   -0.03%     
==========================================
  Files         126      126              
  Lines       19999    19999              
==========================================
- Hits         6612     6608       -4     
- Misses      12329    12334       +5     
+ Partials     1058     1057       -1     
Flag Coverage Δ
integration_tests 20.66% <33.33%> (-0.02%) :arrow_down:
integration_tests_byzantine 9.62% <33.33%> (ø)
integration_tests_gov 9.95% <33.33%> (ø)
integration_tests_grpc 9.93% <33.33%> (ø)
integration_tests_ibc 24.74% <33.33%> (ø)
integration_tests_ledger 9.82% <33.33%> (ø)
integration_tests_slow 9.82% <33.33%> (-0.03%) :arrow_down:
integration_tests_solomachine 9.62% <33.33%> (ø)
integration_tests_upgrade 9.93% <33.33%> (ø)
unit_tests 10.97% <33.33%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
app/app.go 91.05% <33.33%> (-0.73%) :arrow_down: