Title: bpo-38323: Change MultiLoopChildWatcher to install handlers for all the event loops by shreyanavigyan · Pull Request #26574 · python/cpython · GitHub
Open Graph Title: bpo-38323: Change MultiLoopChildWatcher to install handlers for all the event loops by shreyanavigyan · Pull Request #26574 · python/cpython
X Title: bpo-38323: Change MultiLoopChildWatcher to install handlers for all the event loops by shreyanavigyan · Pull Request #26574 · python/cpython
Description: Fix a race condition in MultiLoopChildWatcher. This patch does the following - Creates a list that will keep track of the event loops where the handler has been installed by MultiLoopChildWatcher. In attach_loop and add_child_handler, the handler is installed for the loop using add_signal_handler and the event loop instance is also added to the list. In remove_child_handler, the remove_signal_handler is called on the loop the provided pid is attached to. In close, the remove_signal_handler is called on every loop and the loop is removed from the list. This patch changes the behavior of MultiLoopChildWatcher. The original idea remains the same but with this patch MultiLoopChildWatcher can only be used in the main thread. https://bugs.python.org/issue38323
Open Graph Description: Fix a race condition in MultiLoopChildWatcher. This patch does the following - Creates a list that will keep track of the event loops where the handler has been installed by MultiLoopChildWatcher....
X Description: Fix a race condition in MultiLoopChildWatcher. This patch does the following - Creates a list that will keep track of the event loops where the handler has been installed by MultiLoopChildWatcher....
Opengraph URL: https://github.com/python/cpython/pull/26574
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:6e9e1042-954f-595f-ed5f-864bcc656f0e |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | 9BC0:27A664:237D75:31A902:696A06B0 |
| html-safe-nonce | 4b30158a71ca1d85f4fbc6446c2ffc74c3fd560dc0f8be1b4a6628b305b6f3ba |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI5QkMwOjI3QTY2NDoyMzdENzU6MzFBOTAyOjY5NkEwNkIwIiwidmlzaXRvcl9pZCI6IjQ2MTcwODI3Nzk3NDczNTQyODgiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | b34d2e7013e368875ac2d1ece4f734939a7692acc2aefe8a4f590ef1a87a4eee |
| hovercard-subject-tag | pull_request:663308950 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/python/cpython/pull/26574/files |
| twitter:image | https://avatars.githubusercontent.com/u/74560907?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/74560907?s=400&v=4 |
| og:image:alt | Fix a race condition in MultiLoopChildWatcher. This patch does the following - Creates a list that will keep track of the event loops where the handler has been installed by MultiLoopChildWatcher.... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 578c119ff0247c8b2f2491fbf4fc0395cdf909d4df66598cebdc96ddfc4418dc |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/python/cpython git https://github.com/python/cpython.git |
| octolytics-dimension-user_id | 1525981 |
| octolytics-dimension-user_login | python |
| octolytics-dimension-repository_id | 81598961 |
| octolytics-dimension-repository_nwo | python/cpython |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 81598961 |
| octolytics-dimension-repository_network_root_nwo | python/cpython |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 671c2f67171dbced24284331f3133a613d08c366 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width