Skip to content

Conversation

@pablogsal
Copy link
Member

@pablogsalpablogsal commented Jul 8, 2025

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

@pablogsal
Copy link
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
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

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

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

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