filecoin-project / specs-actors

DEPRECATED Specification of builtin actors, in the form of executable code.
Other
86 stars 102 forks source link

NV15: Speed up the migration #1548

Closed arajasek closed 2 years ago

arajasek commented 2 years ago

The special Sectors migrating logic is as follows:

codecov-commenter commented 2 years ago

Codecov Report

Merging #1548 (7140d56) into master (b1dfad7) will not change coverage. The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1548   +/-   ##
======================================
  Coverage    69.0%   69.0%           
======================================
  Files          73      73           
  Lines        8798    8798           
======================================
  Hits         6079    6079           
  Misses       1858    1858           
  Partials      861     861           
arajasek commented 2 years ago

Rebased on top of latest master (revert of deal label PR)

arajasek commented 2 years ago

Testing update: Ran on mainnet state with and without the cache, same result.

Next steps are a unit test coming from @geoff-vball and maybe writing an unmigration, and testing that.

arajasek commented 2 years ago

Landing this, thanks Steb!