Uh oh!
There was an error while loading. Please reload this page.
GH-146527: Add get_gc_stats function to _remote_debugging - #148071
Conversation
Uh oh!
There was an error while loading. Please reload this page.
sergey-miryanov
commented
Apr 10, 2026
I'm not sure about news entry too. |
sergey-miryanov
commented
Apr 12, 2026
@pablogsal Could you please take a look? |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
a929c0c to
e95d13dCompareThanks a lot for the great work on this @sergey-miryanov! I pushed a set of follow-up commits on top because we are getting close to beta freeze and I wanted to make sure this lands in a solid shape. The follow-ups validate the remote GC stats offset and reject incompatible remote layoutsinstead of accepting short reads, reject a NULL remote GC stats pointer, record timestamps in the free-threaded GC path, and document why On the test side, I renamed the test file to |
Documentation build overview
|
pablogsal
left a comment
There was a problem hiding this comment.
LGTM
Thanks for this @sergey-miryanov this was such a cool idea!
sergey-miryanov
commented
May 2, 2026
Thanks so much, @pablogsal! You did excellent work shaping this into its final form! |
This PR implements second part of the issue. Instead of adding a new module or extending the
gcmodule we add new function to the_remote_debuggingmodule as it is a best place for this.