Skip to content

gh-136692:Improve test cleanup and thread handling in monitoring#136896

Closed
MengAiDev wants to merge 4 commits intopython:mainfrom
MengAiDev:meng
Closed

gh-136692:Improve test cleanup and thread handling in monitoring#136896
MengAiDev wants to merge 4 commits intopython:mainfrom
MengAiDev:meng

Conversation

@MengAiDev
Copy link
Copy Markdown

@MengAiDev MengAiDev commented Jul 21, 2025

  • Replace manual thread alive checks with join() for reliability and efficiency
  • Disable monitoring events after test to prevent resource leaks
  • Fix race conditions by ensuring threads terminate before test completion

…g tests

- Replace manual thread alive checks with `join()` for reliability and efficiency
- Disable monitoring events after test to prevent resource leaks
- Fix race conditions by ensuring threads terminate before test completion
@bedevere-app bedevere-app bot added awaiting review tests Tests in the Lib/test dir labels Jul 21, 2025
@bedevere-app
Copy link
Copy Markdown

bedevere-app bot commented Jul 21, 2025

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@bedevere-app
Copy link
Copy Markdown

bedevere-app bot commented Jul 21, 2025

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@bedevere-app
Copy link
Copy Markdown

bedevere-app bot commented Jul 21, 2025

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@bedevere-app
Copy link
Copy Markdown

bedevere-app bot commented Jul 21, 2025

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@MengAiDev MengAiDev closed this Jul 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant