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

CPU Restriction is unstable #53

Open
Frontrider opened this issue Mar 26, 2018 · 3 comments
Open

CPU Restriction is unstable #53

Frontrider opened this issue Mar 26, 2018 · 3 comments

Comments

@Frontrider
Copy link

as I noted in #52 the CPU control method sometimes throws a RuntineException instead of the expected ScriptCPUAbuseException.

It's a rare occasion (The unit tests sometimes fail for no reason), but I can confirm that it can happen, and should be handled.

@mxro
Copy link
Collaborator

mxro commented Apr 18, 2018

Thank you for reporting this as well. So would you say the fix is to change the unit tests or to try to force throwing a ScriptCPUAbuse exception always? What kind of RuntimeException is triggered? Thank you!

@Frontrider
Copy link
Author

Simply a runtime exception. No other thing is known.

It's extremely rare, only happened once, but it should still be noted, that it has a very slight chance to do that. Might have been something thread related. Like it crashed while it was trying to shut it down.

From what I see, that method should not throw anything else, under normal circumstances. That is what we expect, that is what we test for.

@mxro
Copy link
Collaborator

mxro commented May 12, 2018

Given that this appears only very rarely, I don't think there is anything we can do about it at the moment since we won't be able to pinpoint where the RuntimeException originates from.

In any case, will leave this issue open in case that exception comes up again. Then we could add the stack trace for the RuntimeException and that will maybe help debug it!

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

No branches or pull requests

2 participants