Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve market actor efficiency handling terminated sectors #1498

Closed
anorth opened this issue Jan 15, 2024 · 2 comments · Fixed by #1585
Closed

Improve market actor efficiency handling terminated sectors #1498

anorth opened this issue Jan 15, 2024 · 2 comments · Fixed by #1585
Labels

Comments

@anorth
Copy link
Member

anorth commented Jan 15, 2024

The market actor method on_miner_sectors_terminate inefficiently loads the collection root for both proposals and deal states for each deal in the terminated sector/s. Refactor the state type to load the collection once and then perform operations on the collection, before then saving the collection root.

@jbarnes850
Copy link

Hey @anorth and @rjan90, are you comfortable if I jump in and start working on this?

@jennijuju
Copy link
Member

Hey @anorth and @rjan90, are you comfortable if I jump in and start working on this?

👍🏻

@kt-wawro kt-wawro moved this to In progress in PLDG Cohort 0 Project Board Jul 17, 2024
@rjan90 rjan90 moved this from 🐱 Todo to ⌨️ In Progress in FilOz Oct 22, 2024
@rjan90 rjan90 moved this from Todo to In Progress in nv22 - Track Board Oct 22, 2024
@github-project-automation github-project-automation bot moved this from In progress to Done in PLDG Cohort 0 Project Board Oct 28, 2024
@github-project-automation github-project-automation bot moved this from ⌨️ In Progress to 🎉 Done in FilOz Oct 28, 2024
@rjan90 rjan90 moved this from 🎉 Done to ☑️ Done (Archive) in FilOz Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: ☑️ Done (Archive)
Status: In Progress
3 participants