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

Let the PVF host kill the worker on timeout #6381

Merged
merged 5 commits into from
Dec 6, 2022
Merged

Let the PVF host kill the worker on timeout #6381

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
79cc6ff
Let the PVF host kill the worker on timeout
mrcnski Dec 2, 2022
f035999
Fix comment
mrcnski Dec 2, 2022
4b85076
Fix inaccurate comments; add missing return statement
mrcnski Dec 2, 2022
d0d0c18
Fix a comment
mrcnski Dec 2, 2022
1ee8695
Fix comment
mrcnski Dec 5, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Fix comment
  • Loading branch information
@mrcnski
mrcnski committed Dec 5, 2022
commit 1ee86959367101dfb44406338c13328e9dff1405
6 changes: 2 additions & 4 deletions node/core/pvf/src/worker_common.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -199,10 +199,8 @@ where
}

/// Loop that runs in the CPU time monitor thread on prepare and execute jobs. Continuously wakes up
/// from sleeping and then either sleeps for the remaining CPU time, or kills the process if we
/// exceed the CPU timeout.
///
/// NOTE: Killed processes are detected and cleaned up in `purge_dead`.
/// from sleeping and then either sleeps for the remaining CPU time, or sends back a timeout error
/// if we exceed the CPU timeout.
///
/// NOTE: If the job completes and this thread is still sleeping, it will continue sleeping in the
/// background. When it wakes, it will see that the flag has been set and return.
Expand Down