Removal of the ProveReplicaUpdates method from the miner actor #1151
rvagg
started this conversation in
Enhancements - Technical
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
I'm proposing the removal of the
ProveReplicaUpdates
method (Method 27) from the miner actor.This method has been effectively (not technically) deprecated since the introduction of
ProveReplicaUpdates3
, which provides the same functionality with greater flexibility for handling Snap Deals operations.On-chain analysis indicates this method has not been actively used for at least six months, with storage providers having migrated to the newer
ProveReplicaUpdates3
method. Code to use it was been removed from Lotus Miner for some time and was never implemented in Curio.This proposal follows the same pattern as FIP-0101 and FIP-0103, which proposed removing other superseded methods (
ProveCommitAggregate
andExtendSectorExpiration
respectively).The proposed removal would help reduce technical debt and simplify the protocol without impacting current operations, as all Snap Deals functionality is already covered by
ProveReplicaUpdates3
. Storage providers and tooling have already migrated to using the newer method, making this a safe change.It's important to note that this FIP does not remove Snap Deals functionality - it only removes the redundant legacy method while preserving full functionality through the more flexible
ProveReplicaUpdates3
method.FIP pull request: #1152
Beta Was this translation helpful? Give feedback.
All reactions