Skip to content
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

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

rruuaanng
Copy link
Contributor

@rruuaanng rruuaanng commented Oct 25, 2024

@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.

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

Successfully merging this pull request may close these issues.

2 participants