-
-
Notifications
You must be signed in to change notification settings - Fork 31.9k
gh-110752: Reset ceval.eval_breaker
to 0 in interpreter_clear
#110753
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
gh-110752: Reset ceval.eval_breaker
to 0 in interpreter_clear
#110753
Conversation
Looks good |
|
|
I tried to read the code and I believe at this point all the threads should be done so we don't need atomic operations.
Also no test was added for now. The test requires interpreter finalization so it needs to be in its own process.