Skip to content

PEP 805: Safe Parallel Python - #4579

Closed
markshannon wants to merge 19 commits into
python:mainfrom
markshannon:safe-parallel
Closed

PEP 805: Safe Parallel Python#4579
markshannon wants to merge 19 commits into
python:mainfrom
markshannon:safe-parallel

Conversation

@markshannon

@markshannon markshannon commented Sep 8, 2025 •
edited by hugovk
Loading

Copy link
Copy Markdown
Member

StanFromIreland
StanFromIreland reviewed Sep 8, 2025
View reviewed changes
Comment thread peps/pep-0804.rst Outdated
@StanFromIreland StanFromIreland added the new-pep A new draft PEP submitted for initial review label Sep 8, 2025
@AA-Turner AA-Turner changed the title Safe parallel PEP 805: Safe Parallel Python Sep 8, 2025
AA-Turner
AA-Turner reviewed Sep 8, 2025
View reviewed changes

@AA-Turner AA-Turner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two initial changes to fix the headers.

When this is ready for review, please could you wrap to 79 columns?

A

Comment thread peps/pep-0805.rst Outdated
Comment thread peps/pep-0805.rst Outdated
@markshannon

markshannon commented Sep 8, 2025

Copy link
Copy Markdown
Member Author

I don't understand the error: /home/runner/work/peps/peps/peps/pep-0805.rst:484: (WARNING/2) Inline emphasis start-string without end-string.

Line 484 is in a block of C code.

brianschubert
brianschubert reviewed Sep 8, 2025
View reviewed changes
Comment thread peps/pep-0805.rst
Comment thread peps/pep-0805.rst
brianschubert
brianschubert reviewed Sep 8, 2025
View reviewed changes
Comment thread peps/pep-0805.rst Outdated
@brianschubert

This comment was marked as resolved.

Sign in to view
JelleZijlstra
JelleZijlstra reviewed Sep 8, 2025
View reviewed changes

@JelleZijlstra JelleZijlstra left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a few typos

Comment thread peps/pep-0805.rst Outdated
Comment thread peps/pep-0805.rst Outdated
Comment thread peps/pep-0805.rst Outdated
Comment thread peps/pep-0805.rst Outdated
@markshannon @JelleZijlstra
Apply suggestions from code review
2fa8f80 
Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>
@markshannon

markshannon commented Sep 8, 2025

Copy link
Copy Markdown
Member Author

Thanks @JelleZijlstra

sobolevn
sobolevn reviewed Sep 9, 2025
View reviewed changes
Comment thread peps/pep-0805.rst Outdated
hugovk
hugovk reviewed Sep 9, 2025
View reviewed changes
Comment thread peps/pep-0805.rst
Comment thread peps/pep-0805.rst Outdated
Comment thread peps/pep-0805.rst Outdated
Comment thread peps/pep-0805.rst Outdated
Comment thread peps/pep-0805.rst Outdated
Comment thread peps/pep-0805.rst Outdated
Comment thread peps/pep-0805.rst Outdated
Comment thread peps/pep-0805.rst Outdated
Comment thread peps/pep-0805.rst Outdated
Comment thread peps/pep-0805.rst Outdated
TrulsHenriksson
TrulsHenriksson reviewed Sep 25, 2025
View reviewed changes
Comment thread peps/pep-0805.rst Outdated
@pablogsal @TrulsHenriksson
Update peps/pep-0805.rst
78261bd 
Co-authored-by: Truls Henriksson <trulshenriksson@gmail.com>
@ncoghlan

ncoghlan commented Oct 7, 2025

Copy link
Copy Markdown
Contributor

As far as prior art goes, it may be worth adding a brief mention of Rust's threading model as well, as that adopts a similar "thread local by default" approach to minimising the likelihood of encountering data races in multi-threaded programs.

hugovk
hugovk reviewed May 7, 2026
View reviewed changes
Comment thread peps/pep-0805.rst Outdated
@markshannon
markshannon marked this pull request as ready for review July 8, 2026 14:15
@markshannon
markshannon requested a review from a team as a code owner July 8, 2026 14:15
markshannon and others added 2 commits July 8, 2026 15:15
@markshannon @hugovk
Update peps/pep-0805.rst
1ce2c5a 
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
@markshannon
Merge branch 'main' into safe-parallel
df91af4
@read-the-docs-community

read-the-docs-community Bot commented Jul 8, 2026

Copy link
Copy Markdown

Documentation build overview

📚 pep-previews | 🛠️ Build #33497350 | 📁 Comparing df91af4 against latest (4813899)

  🔍 Preview build  

5 files changed · + 1 added · ± 4 modified

+ Added

± Modified

@markshannon

markshannon commented Aug 20, 2026

Copy link
Copy Markdown
Member Author

Thanks for the reviews.
This PEP has undergone considerable changes and development, so I've opened a new PR for it: #5101

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sergey-miryanov sergey-miryanov sergey-miryanov left review comments
@JelleZijlstra JelleZijlstra JelleZijlstra left review comments
@hugovk hugovk hugovk left review comments
@sobolevn sobolevn sobolevn left review comments
@AA-Turner AA-Turner AA-Turner left review comments
@brianschubert brianschubert brianschubert left review comments
@StanFromIreland StanFromIreland StanFromIreland left review comments
+1 more reviewer
@TrulsHenriksson TrulsHenriksson TrulsHenriksson left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

new-pep A new draft PEP submitted for initial review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.