PR feedback on looping for process death

csm10495 · csm10495 · commit 4c2f35f96d20 · 2025-03-04T11:48:26.000-08:00
diff --git a/Lib/test/test_concurrent_futures/test_process_pool.py b/Lib/test/test_concurrent_futures/test_process_pool.py
@@ -334,8 +334,9 @@ def test_force_shutdown_workers_stops_pool(self, function_name):
             # ...  that seems a bit buggy.
             # We need it dead before ending the test to ensure it doesn't
             # get marked as an ENV CHANGE due to living child process.
-            while worker_process.is_alive():
-                time.sleep(0.1)
+            for _ in support.sleeping_retry(support.SHORT_TIMEOUT):
+                if not worker_process.is_alive():
+                    break
 
 
 create_executor_tests(globals(), ProcessPoolExecutorTest,
