Closed Trisfald closed 1 month ago
Attention: Patch coverage is 0%
with 31 lines
in your changes missing coverage. Please review.
Project coverage is 71.53%. Comparing base (
0e6179a
) to head (a0c945f
). Report is 4 commits behind head on master.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
First part of cleanup of legacy resharding V2 code.
Notes:
resharding
tool_v2
suffixThis is just the beginning.. many other things must be removed as well:
StateSnapshotType::ForReshardingOnly
StateChangeCauseView::Resharding
StateChangesForResharding
InvalidSplitShardsIds
ShardSyncStatus::{ReshardingScheduling, ReshardingApplying}
ReshardingResult
ApplyResultForResharding
State_sync::resharding_state_roots
chain.rs
ShardUpdateReason::Resharding
chain/src/store/mod.rs
I think it's better to tackle the other items in different PRs, but I don't have a strong opinion