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

[prover] Sort proving jobs by discrete priorities #7925

Closed
Tracked by #7346
spalladino opened this issue Aug 12, 2024 · 0 comments
Closed
Tracked by #7346

[prover] Sort proving jobs by discrete priorities #7925

spalladino opened this issue Aug 12, 2024 · 0 comments
Labels
S-needs-discussion Status: Still needs more discussion before work can start.

Comments

@spalladino
Copy link
Collaborator

spalladino commented Aug 12, 2024

In #7844 we added an epochNumber for sorting jobs. However, this makes it difficult to implement with a fifo queue with no support for priorities. Instead, we could use a discrete set of priorities (eg level in the proof tree) to sort jobs, so this can be implemented with multiple fifo queues.

Alternatively, we can use something that is not a queue for sorting jobs, such as a db, given we won't have such a large tx throughput. If we do so, and we enshrine epoch numbers as a priority, then this issue turns into addressing @alexghr's comments on the original PR to make this priority mandatory and part of the proving job itself.

Needs further discussion.

@github-project-automation github-project-automation bot moved this to Todo in A3 Aug 12, 2024
@spalladino spalladino changed the title Sort proving jobs by discrete priorities [prover] Sort proving jobs by discrete priorities Aug 27, 2024
@spalladino spalladino added the S-needs-discussion Status: Still needs more discussion before work can start. label Sep 11, 2024
@PhilWindle PhilWindle closed this as not planned Won't fix, can't repro, duplicate, stale Oct 29, 2024
@github-project-automation github-project-automation bot moved this from Todo to Done in A3 Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-needs-discussion Status: Still needs more discussion before work can start.
Projects
Archived in project
Development

No branches or pull requests

2 participants