Skip to content

do not force shutdown the executor in the cleanup - #164

Merged
basnijholt merged 1 commit into
stable-0.7from
no_force_executor_shutdown
Mar 21, 2019
Merged

do not force shutdown the executor in the cleanup#164
basnijholt merged 1 commit into
stable-0.7from
no_force_executor_shutdown

Conversation

@basnijholt

@basnijholtbasnijholt commented Mar 18, 2019

Copy link
Copy Markdown
Member

See #156

This prevent the 'OSError: handle is closed' after the runner
is closed in Python 3.7

@basnijholt
basnijholtforce-pushed the no_force_executor_shutdown branch 2 times, most recently from 986a84a to 79c4269CompareMarch 18, 2019 08:27
@basnijholtbasnijholt mentioned this pull request Mar 18, 2019
@basnijholt
basnijholtforce-pushed the no_force_executor_shutdown branch from 79c4269 to 1f6bf6fCompareMarch 21, 2019 16:30
See #156
This prevent the 'OSError: handle is closed' after the runner
is closed in Python 3.7
@basnijholt
basnijholtforce-pushed the no_force_executor_shutdown branch from 1f6bf6f to 069964eCompareMarch 21, 2019 16:36
@basnijholt
basnijholt changed the base branch from master to stable-0.7March 21, 2019 16:36
@basnijholt
basnijholt removed the request for review from akhmerovMarch 21, 2019 16:38
@basnijholt
basnijholt merged commit 142bf49 into stable-0.7Mar 21, 2019
@basnijholt
basnijholt deleted the no_force_executor_shutdown branch March 21, 2019 16:46
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@basnijholt