Skip to content

gh-91048: Revert the memory cache removal for remote debugging - #136440

Merged
pablogsal merged 2 commits into
python:mainfrom
pablogsal:cache
Jul 9, 2025
Merged

gh-91048: Revert the memory cache removal for remote debugging#136440
pablogsal merged 2 commits into
python:mainfrom
pablogsal:cache

Conversation

@pablogsal

@pablogsalpablogsal commented Jul 8, 2025

Copy link
Copy Markdown
Member

Some users have complained that removing the cache makes async introspection slower

@pablogsal

Copy link
Copy Markdown
MemberAuthor

Note: this partially reverts the cache removal of #136005 so this code was in previous betas, just reverting back to it.

@pablogsalpablogsal changed the title gh-91048: Reintroduce the memory cache for remote debugginggh-91048: Revert the memory cache removal for remote debuggingJul 8, 2025
@pablogsal

Copy link
Copy Markdown
MemberAuthor

Confirmed the regression:

Current:

Real-time sampling stats: Mean: 4498.8Hz (222.28µs) Min: 83.1Hz (12029.75µs) Max: 5065.9Hz (197.40µs) Samples: 7503^CTraceback (most recent call last):

Before the cache removal:

Real-time sampling stats: Mean: 10835.8Hz (92.29µs) Min: 81.9Hz (12207.85µs) Max: 11610.2Hz (86.13µs) Samples: 7413

@pablogsal
pablogsal enabled auto-merge (squash) July 8, 2025 23:55
@pablogsal
pablogsal merged commit 77d25e5 into python:mainJul 9, 2025
39 checks passed
@pablogsal
pablogsal deleted the cache branch July 9, 2025 00:31
@miss-islington-app

Copy link
Copy Markdown

Thanks @pablogsal for the PR 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 9, 2025
…ythonGH-136440)
(cherry picked from commit 77d25e5)
Co-authored-by: Pablo Galindo Salgado <Pablogsal@gmail.com>
pythongh-91048: Reintroduce the memory cache for remote debugging
@bedevere-app

Copy link
Copy Markdown

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

@bedevere-appbedevere-appBot removed the needs backport to 3.14 bugs and security fixes label Jul 9, 2025
pablogsal added a commit that referenced this pull request Jul 9, 2025
…GH-136440) (#136443)
gh-91048: Revert the memory cache removal for remote debugging (GH-136440)
(cherry picked from commit 77d25e5)
gh-91048: Reintroduce the memory cache for remote debugging
Co-authored-by: Pablo Galindo Salgado <Pablogsal@gmail.com>
@bedevere-bot

This comment has been minimized.

AndPuQing pushed a commit to AndPuQing/cpython that referenced this pull request Jul 11, 2025
Pranjal095 pushed a commit to Pranjal095/cpython that referenced this pull request Jul 12, 2025
picnixz pushed a commit to picnixz/cpython that referenced this pull request Jul 13, 2025
taegyunkim pushed a commit to taegyunkim/cpython that referenced this pull request Aug 4, 2025
Agent-Hellboy pushed a commit to Agent-Hellboy/cpython that referenced this pull request Aug 19, 2025
kumaraditya303 pushed a commit to miss-islington/cpython that referenced this pull request Sep 9, 2025
…ugging (pythonGH-136440) (python#136443)
pythongh-91048: Revert the memory cache removal for remote debugging (pythonGH-136440)
(cherry picked from commit 77d25e5)
pythongh-91048: Reintroduce the memory cache for remote debugging
Co-authored-by: Pablo Galindo Salgado <Pablogsal@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@pablogsal@bedevere-bot