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

[3.13] gh-75876: Correct the memory use declared by some bigmem tests (GH-155307) by miss-islington · Pull Request #155314 · python/cpython · GitHub

Repository navigation

[3.13] gh-75876: Correct the memory use declared by some bigmem tests (GH-155307) - #155314

Merged
serhiy-storchaka merged 1 commit into
python:3.13from
miss-islington:backport-e469fa9-3.13
Aug 7, 2026
Merged

serhiy-storchaka merged 1 commit into
python:3.13from
miss-islington:backport-e469fa9-3.13

Conversation

miss-islington commented Aug 7, 2026 •
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Contributor

Measured with a limited address space and the peak RSS of the process:

  • TupleTest.test_repeat_large_2 keeps 3 tuples of pointers alive, as its
    siblings do, but declared the count for a 32-bit build.
  • The md5 tests hash the data and a io.BytesIO() copy of it.
  • testDecompress4G keeps the data, the compressed and the decompressed
    bytes alive, and the decompressor grows its output buffer.
  • test_huge_string_builtins uses 3 times the string size, not 2.
  • test_large_subn uses a bit more than the 18 times it declared.
    (cherry picked from commit e469fa9)

Co-authored-by: Serhiy Storchaka storchaka@gmail.com
Co-authored-by: Claude Opus 5 (1M context) noreply@anthropic.com

…ythonGH-155307)

Measured with a limited address space and the peak RSS of the process:

* TupleTest.test_repeat_large_2 keeps 3 tuples of pointers alive, as its
  siblings do, but declared the count for a 32-bit build.
* The md5 tests hash the data and a io.BytesIO() copy of it.
* testDecompress4G keeps the data, the compressed and the decompressed
  bytes alive, and the decompressor grows its output buffer.
* test_huge_string_builtins uses 3 times the string size, not 2.
* test_large_subn uses a bit more than the 18 times it declared.
(cherry picked from commit e469fa9)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
bedevere-app Bot added the tests Tests in the Lib/test dir label Aug 7, 2026
serhiy-storchaka enabled auto-merge (squash) August 7, 2026 06:38
serhiy-storchaka merged commit d8c20a5 into python:3.13 Aug 7, 2026
43 checks passed
miss-islington deleted the backport-e469fa9-3.13 branch August 7, 2026 06:56
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL