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

[3.14] gh-146452: Improve locking granularity in pickle's batch_dict_… by scopreon · Pull Request #150062 · python/cpython · GitHub

Repository navigation

[3.14] gh-146452: Improve locking granularity in pickle's batch_dict_… - #150062

Merged
JelleZijlstra merged 1 commit into
python:3.14from
scopreon:backport-57a0e57-3.14
May 19, 2026
Merged

JelleZijlstra merged 1 commit into
python:3.14from
scopreon:backport-57a0e57-3.14

Conversation

scopreon commented May 19, 2026 •
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Contributor

…exact and fix race condition (GH-150025)

Remove assertion that could fail in rare race condition.

Replace the coarse critical section wrapping the entire function with fine-grained sections covering only PyDict_Next + Py_INCREF. Also handle PyDict_Next returning 0 in the single-item fast path. (cherry picked from commit 57a0e57)

…_dict_exact and fix race condition (pythonGH-150025)

Remove assertion that could fail in rare race condition.

Replace the coarse critical section wrapping the entire function with
fine-grained sections covering only PyDict_Next + Py_INCREF.
Also handle PyDict_Next returning 0 in the single-item fast path.
(cherry picked from commit 57a0e57)

Co-authored-by: Saul Cooperman <58375603+scopreon@users.noreply.github.com>

bedevere-app Bot commented May 22, 2026

Copy link
Copy Markdown

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

kumaraditya303 pushed a commit that referenced this pull request May 23, 2026
…e's batch_dict_… (#150062)" (#150263)

Revert "[3.14] gh-146452: Improve locking granularity in pickle's batch_dict_… (#150062)"

This reverts commit 1243cd6.
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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL