FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-159293: Restore the recursion limit after the free-threading race test by zolaski333 · Pull Request #159311 · python/cpython · GitHub

  1. python
  2. cpython

Repository navigation

gh-159293: Restore the recursion limit after the free-threading race test - #159311

Merged
freakboy3742 merged 1 commit into
python:mainfrom
zolaski333:codex/gh-159293-restore-recursion-limit
Oct 14, 2026
ConversationCommits (1)ChecksFiles changed
Merged

gh-159293: Restore the recursion limit after the free-threading race test#159311
freakboy3742 merged 1 commit into
python:mainfrom
zolaski333:codex/gh-159293-restore-recursion-limit

Conversation

zolaski333 commented Oct 13, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

test_racing_recursion_limit leaves the interpreter's recursion limit at 199. When tests share a process, the Python lru_cache recursion test and 68 pickle cases subsequently raise RecursionError.

Register a cleanup to restore the original recursion limit even if the race test fails. This preserves the existing concurrency test and follows the TestCase.addCleanup pattern used elsewhere in the suite.

Fixes gh-159293.

Validation on Windows x64, CPython main at 50082b8d893, free-threaded Release build with the GIL disabled:

  • Before the fix: race test passes but changes the limit from 1000 to 199; TestLRUPy.test_lru_recursion has one error and test_pickle has 68 errors.
  • After the fix: limit remains 1000; the same 1086 tests complete without errors (42 skipped).
  • PCbuild/amd64t/python.exe -X dev -m test --single-process --no-randomize test_free_threading test_functools test_pickle: SUCCESS, 1674 tests run, 52 skipped; 55 test files passed and six platform-specific files skipped.
  • Tools/patchcheck/patchcheck.py and git diff --check pass.

bedevere-app Bot mentioned this pull request Oct 13, 2026
Closed
bedevere-app Bot added awaiting review tests Tests in the Lib/test dir labels Oct 13, 2026
aisk reviewed Oct 13, 2026
View reviewed changes
Comment thread Misc/NEWS.d/next/Tests/2026-10-13-13-34-00.gh-issue-159293.Lr8q2N.rst
@@ -0,0 +1,2 @@
Restore the recursion limit after the free-threading recursion-limit race test

aisk Oct 13, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Test only changes don't requires a news entry file, please remove it.

terryjreedy Oct 13, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

'Tests' is one of the blurb options, for test-only PRs, and this was put in the Tests section.

aisk Oct 14, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

The DevGuide has some wording on this: https://github.com/python/devguide/blob/b89ba85ad4d34984812210f6460d619565839971/getting-started/pull-request-lifecycle.rst?plain=1#L218-L219. So does this mean a NEWS entry is just optional for test-only changes?

However, I've seen other test-only PRs where the author was asked to remove the NEWS entry:

aisk added the skip news label Oct 13, 2026
freakboy3742 added needs backport to 3.14 bugs and security fixes needs backport to 3.15 bugs and security fixes labels Oct 14, 2026
freakboy3742 approved these changes Oct 14, 2026
View reviewed changes

freakboy3742 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Thanks for the fix!

freakboy3742 merged commit aeff4cb into python:main Oct 14, 2026
66 checks passed

miss-islington-app Bot commented Oct 14, 2026

Copy link
Copy Markdown

Thanks @zolaski333 for the PR, and @freakboy3742 for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14, 3.15.
🐍🍒⛏🤖

bedevere-app Bot commented Oct 14, 2026

Copy link
Copy Markdown

GH-159389 is a backport of this pull request to the 3.15 branch.

bedevere-app Bot removed the needs backport to 3.15 bugs and security fixes label Oct 14, 2026

bedevere-app Bot commented Oct 14, 2026

Copy link
Copy Markdown

GH-159390 is a backport of this pull request to the 3.14 branch.

bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label Oct 14, 2026
freakboy3742 mentioned this pull request Oct 14, 2026
Merged
freakboy3742 pushed a commit that referenced this pull request Oct 14, 2026
[3.14] gh-159293: Restore the recursion limit after the free-threadin…
3fa2a37 
…g race test (GH-159311) (#159390)

Adds a cleanup to ensure interpreter state is restored after testing recursion
limit changes.
(cherry picked from commit aeff4cb)

Co-authored-by: Noah <57315647+zolaski333@users.noreply.github.com>
freakboy3742 pushed a commit that referenced this pull request Oct 14, 2026
[3.15] gh-159293: Restore the recursion limit after the free-threadin…
3396c34 
…g race test (GH-159311) (#159389)

Adds a cleanup to ensure interpreter state is restored after testing recursion
limit changes.
(cherry picked from commit aeff4cb)

Co-authored-by: Noah <57315647+zolaski333@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

aisk aisk left review comments
terryjreedy terryjreedy left review comments
freakboy3742 freakboy3742 approved these changes

Assignees

No one assigned

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Test ordering error with free-threaded builds

4 participants

Back | FazBrowse Home | New Git URL