Skip to content
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.

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

Use concurrent.futures for concurrency with thread and process #87

Merged
merged 9 commits into from
Feb 11, 2020

Conversation

miurahr
Copy link
Owner

@miurahr miurahr commented Feb 9, 2020

Signed-off-by: Hiroshi Miura [email protected]

@miurahr miurahr changed the title Use concurrent.futures for concurrency with thread and process WIP:Use concurrent.futures for concurrency with thread and process Feb 9, 2020
@miurahr miurahr force-pushed the topic-multiprocess branch from 024f139 to a31e321 Compare February 9, 2020 12:17
- combination with thread pool for downloading and
  process pool for extraction.

Signed-off-by: Hiroshi Miura <[email protected]>
- Start extraction when download completed
- Add download completion log

Signed-off-by: Hiroshi Miura <[email protected]>
Signed-off-by: Hiroshi Miura <[email protected]>
@miurahr miurahr force-pushed the topic-multiprocess branch from 6d50715 to 43c3b7a Compare February 9, 2020 23:12
@miurahr miurahr changed the title WIP:Use concurrent.futures for concurrency with thread and process Use concurrent.futures for concurrency with thread and process Feb 10, 2020
- use perf_counter() for a total elapsed time.
- use process_time() for individual extractions.

Signed-off-by: Hiroshi Miura <[email protected]>
Signed-off-by: Hiroshi Miura <[email protected]>
@miurahr miurahr merged commit bbb39c0 into master Feb 11, 2020
@miurahr miurahr deleted the topic-multiprocess branch February 11, 2020 01:59
miurahr added a commit that referenced this pull request Feb 28, 2020
- Use multiprocessing.dummy.Pool()
- Related issue #98

Signed-off-by: Hiroshi Miura <[email protected]>
miurahr added a commit that referenced this pull request Mar 24, 2020
* Revert a part of #87 modification

- Use multiprocessing.dummy.Pool()
- Related issue #98 
- try to fix #114

Signed-off-by: Hiroshi Miura <[email protected]>
Sign up for free to 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.

A great amount of time for the win64_mingw73 arch
1 participant