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

Allow setting task time limits #1

Closed
Vectorial1024 opened this issue Dec 5, 2024 · 3 comments · Fixed by #11
Closed

Allow setting task time limits #1

Vectorial1024 opened this issue Dec 5, 2024 · 3 comments · Fixed by #11
Labels
enhancement New feature or request

Comments

@Vectorial1024
Copy link
Owner

Along with this, also include a way to catch "process timed out" events to let the task creator specify on-time-out actions.

@Vectorial1024 Vectorial1024 added the enhancement New feature or request label Dec 5, 2024
@Vectorial1024
Copy link
Owner Author

Experiments show that Process::timeout() is ignored when using Process::start(): the started process will still run until it finishes. We need custom handling of task time limits.

@Vectorial1024
Copy link
Owner Author

To clarify, we are talking about real time, not CPU time. This will include all the time spent on sleeping and waiting that normally may not be counted by PHP (Unix). This has irl value where the overall timing is important.

@Vectorial1024 Vectorial1024 linked a pull request Jan 3, 2025 that will close this issue
@Vectorial1024
Copy link
Owner Author

Blunder for forgetting to link the issue/PR first; closing as done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant