filecoin-project / specs-actors

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

Batch miner state mutations #583

Open ZenGround0 opened 4 years ago

ZenGround0 commented 4 years ago

Filing follow on from this conversation

Batching state mutations will eliminate redundant hamt/amt loading and storing.

anorth commented 4 years ago

Not just in ConfirmSectorProofsValid, but perhaps other places too: audit all uses of PutSector, etc.

Unfortunately PreCommitSector does not currently have batch parameters.

anorth commented 4 years ago

I think we have done many of these. @ZenGround0 @aarshkshah1992 are you aware of more outstanding? Have we examined this code sufficiently recently that we've probably found them all?

@ZenGround0 I will assign this to @aarshkshah1992 to review and close out.