Skip to content

bpo-38858: Add init_interp_main() subfunction #17347

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

Jump to bottom
Merged
merged 1 commit into from
Nov 22, 2019
Merged

bpo-38858: Add init_interp_main() subfunction #17347

merged 1 commit into from
Nov 22, 2019

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Nov 22, 2019 •
edited by bedevere-bot
Loading

Fix new_interpreter() error handling: undo it all if status is an
exception.

https://bugs.python.org/issue38858

@vstinner
bpo-38858: Add init_interp_main() subfunction
ee46562 
Fix new_interpreter() error handling: undo it all if status is an
exception.
@vstinner
Copy link
Member Author

Travis CI failed with ENV_CHANGED. It's unrelated to my change. See: https://bugs.python.org/issue38546

test_map_timeout (test.test_concurrent_futures.ProcessPoolForkProcessPoolExecutorTest) ... 6.17s ok

test_max_workers_negative (test.test_concurrent_futures.ProcessPoolForkProcessPoolExecutorTest) ... 0.16s ok

test_max_workers_too_large (test.test_concurrent_futures.ProcessPoolForkProcessPoolExecutorTest) ... skipped 'Windows-only process limit'

test_no_stale_references (test.test_concurrent_futures.ProcessPoolForkProcessPoolExecutorTest) ... 0.18s ok

test_ressources_gced_in_workers (test.test_concurrent_futures.ProcessPoolForkProcessPoolExecutorTest) ... 1.47s ok

test_shutdown_race_issue12456 (test.test_concurrent_futures.ProcessPoolForkProcessPoolExecutorTest) ... 0.14s ok

test_submit (test.test_concurrent_futures.ProcessPoolForkProcessPoolExecutorTest) ... 0.14s ok

test_submit_keyword (test.test_concurrent_futures.ProcessPoolForkProcessPoolExecutorTest) ... 0.14s
Warning -- reap_children() reaped child process 23508
ok

@vstinner vstinner merged commit b005136 into python:master Nov 22, 2019
@vstinner vstinner deleted the init_interp_main branch November 22, 2019 16:52
jacobneiltaylor pushed a commit to jacobneiltaylor/cpython that referenced this pull request Dec 5, 2019
@vstinner
bpo-38858: Add init_interp_main() subfunction (pythonGH-17347)
c84d0c5 
Fix new_interpreter() error handling: undo it all if status is an
exception.
shihai1991 pushed a commit to shihai1991/cpython that referenced this pull request Jan 31, 2020
@vstinner @shihai1991
bpo-38858: Add init_interp_main() subfunction (pythonGH-17347)
6e3807f 
Fix new_interpreter() error handling: undo it all if status is an
exception.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
skip news
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@vstinner @the-knights-who-say-ni @bedevere-bot