Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

gh-125927: Add docs description for daemon thread #125961

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 7 commits into
base: main
Choose a base branch
Loading
from

Conversation

rruuaanng
Copy link
Contributor

@rruuaanng rruuaanng commented Oct 25, 2024

Doc/library/threading.rst Outdated Show resolved Hide resolved
@ZeroIntensity
Copy link
Member

I think this looks fine, but I'm assuming that Eric wanted something a bit more in depth judging by the original issue, so I'm going to refrain from approving for now.

@rruuaanng
Copy link
Contributor Author

Actually, the focus is on highlighting the differences between Python threads and other threads. This clearly illustrates the unique aspects of Python daemon threads, and I believe Eric won’t have any objections to this.

@ZeroIntensity
Copy link
Member

Well, IIUC, part the motivation was to document some of the concerns about daemon threads as well. This sort of tells you what to do with them, but not what's wrong with them.

@rruuaanng
Copy link
Contributor Author

So in the description, I described that join should be used to solve it.

@ZeroIntensity
Copy link
Member

That's not that useful though, because the whole point of a daemon thread is that you don't want it to join.

@erlend-aasland erlend-aasland marked this pull request as draft November 5, 2024 10:56
@rruuaanng
Copy link
Contributor Author

That's not that useful though, because the whole point of a daemon thread is that you don't want it to join

I'm not sure if the new change is appropriate, but it seems to avoid emphasizing the need to call .join(). But, right, daemon threads generally don't require waiting for their completion, but it still allows the user to call it on them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation in the Doc dir skip news
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

2 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.