8000 gh-88887: Cleanup `multiprocessing.resource_tracker.ResourceTracker` upon deletion by luccabb · Pull Request #130429 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

gh-88887: Cleanup multiprocessing.resource_tracker.ResourceTracker upon deletion #130429

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.

Sign up for GitHub

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

Merged
merged 21 commits into from
Mar 20, 2025
Merged

gh-88887: Cleanup multiprocessing.resource_tracker.ResourceTracker upon deletion #130429

Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
069fb0c
fix resource tracker leak
luccabb Feb 21, 2025
5e33828
try block
luccabb Feb 21, 2025
0586696
add comment to resource tracker __del__
luccabb Feb 21, 2025
eeef2e4
blurb add
luccabb Feb 21, 2025
b3115d2
Merge branch 'main' into fix_resource_tracker_leak
luccabb Feb 21, 2025
cf3d546
Merge branch 'main' of github.com:luccabb/cpython into fix_resource_t…
luccabb Feb 24, 2025
933f70c
catch specific exceptions
luccabb Feb 24, 2025
d4e08c5
Merge branch 'fix_resource_tracker_leak' of github.com:luccabb/cpytho…
luccabb Feb 24, 2025
799ee32
Update Lib/multiprocessing/resource_tracker.py
luccabb Feb 24, 2025
8e43a83
fixing _stop for None pids
luccabb Mar 6, 2025
f9a53a7
pull
luccabb Mar 6, 2025
805ad1d
remove :77 typo
luccabb Mar 6, 2025
5d6c9b3
add extra line back
luccabb Mar 6, 2025
c8e2704
catch AttributeError only and return early from _stop if _pid is None
luccabb Mar 6, 2025
9fd5872
fix lint
luccabb Mar 6, 2025
aaa8378
injecting os methods on _stop
luccabb Mar 7, 2025
dc2d578
Update Lib/multiprocessing/resource_tracker.py comment
gpshead Mar 7, 2025
9847342
non blocking acquire for __del__
luccabb Mar 9, 2025
58b137a
non blocking acquire for __del__
luccabb Mar 9, 2025
d497dee
release only if acquired
luccabb Mar 10, 2025
ba42406
always release + changelog
luccabb Mar 19, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
catch specific exceptions
  • Loading branch information
@luccabb
luccabb committed Feb 24, 2025
commit 933f70c8199b049196742ed0507d551f9608d69c
2 changes: 1 addition & 1 deletion Lib/multiprocessing/resource_tracker.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -218,7 +218,7 @@ def __del__(self):
# see https://github.com/python/cpython/issues/88887
try:
self._stop()
except:
except (OSError, TypeError, AttributeError) as e:
pass


Expand Down
0