Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Kill util::jobs framework #4259

Closed
4 tasks done
rphmeier opened this issue Nov 11, 2021 · 0 comments · Fixed by #5600
Closed
4 tasks done

Kill util::jobs framework #4259

rphmeier opened this issue Nov 11, 2021 · 0 comments · Fixed by #5600
Assignees
Labels
I8-refactor Code needs refactoring. T4-parachains_engineering This PR/Issue is related to Parachains performance, stability, maintenance. T5-parachains_protocol This PR/Issue is related to Parachains features and protocol changes.

Comments

@rphmeier
Copy link
Contributor

rphmeier commented Nov 11, 2021

It did not anticipate things like #3779, where subsystems would need to have "jobs" communicate or coordinate between each other. This could be done in a few parts, where we'd first migrate over the subsystems which use the framework, and then remove the framework entirely. It's not particularly useful code, it adds 'magic' to subsystems, and constitutes an extra maintenance burden.

  • Backing
  • bitfield-signing
  • provisioner
  • remove all JobSubsystem code
@rphmeier rphmeier added the I8-refactor Code needs refactoring. label Nov 11, 2021
@ordian ordian moved this to Implemented in Parachains-core Aug 16, 2022
@ordian ordian added T5-parachains_protocol This PR/Issue is related to Parachains features and protocol changes. T4-parachains_engineering This PR/Issue is related to Parachains performance, stability, maintenance. labels Aug 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
I8-refactor Code needs refactoring. T4-parachains_engineering This PR/Issue is related to Parachains performance, stability, maintenance. T5-parachains_protocol This PR/Issue is related to Parachains features and protocol changes.
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants