-
Notifications
You must be signed in to change notification settings - Fork 85
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
feat(raiko): put the tasks that cannot run in parallel into pending list #358
base: main
Are you sure you want to change the base?
Conversation
smtmfft
commented
Sep 5, 2024
Signed-off-by: smtmfft <smtm@taiko.xyz>
Co-authored-by: Petar Vujović <petarvujovic98@gmail.com>
Co-authored-by: Petar Vujović <petarvujovic98@gmail.com>
opts, | ||
chain_specs, | ||
running_tasks, | ||
pending_tasks, | ||
receiver, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In Rust, we usually use rx and tx to replace receiver and sender.
@@ -146,6 +146,7 @@ pub struct ProverState { | |||
pub enum Message { | |||
Cancel(TaskDescriptor), | |||
Task(ProofRequest), | |||
TaskComplete(ProofRequest), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can use oneshot channel to notify the watcher the result of tasks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you mean notify one-shot channel & let it send task in??