aronisstav / Concuerror

OFFICIAL REPOSITORY MOVED:
https://github.com/parapluu/Concuerror
BSD 2-Clause "Simplified" License
1 stars 0 forks source link

Change spawn #34

Closed PanagiotisFytas closed 6 years ago

PanagiotisFytas commented 6 years ago

Summary

This will change the way the scheduler spawns new processes. Specifically, a default of 50 process are getting spawned beforehand and when a new spawn is required the scheduler gets the new process from a pool of idle processes.

Checklist

codecov-io commented 6 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@6f09a23). Click here to learn what that means. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #34   +/-   ##
=========================================
  Coverage          ?   93.62%           
=========================================
  Files             ?       12           
  Lines             ?     3406           
  Branches          ?        0           
=========================================
  Hits              ?     3189           
  Misses            ?      217           
  Partials          ?        0
Impacted Files Coverage Δ
src/concuerror_generator.erl 100% <100%> (ø)
src/concuerror_callback.erl 92.56% <100%> (ø)
src/concuerror_options.erl 97.83% <100%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 6f09a23...ee3f072. Read the comment docs.

aronisstav commented 6 years ago

Excellent work! 👍 👍 👍

What I did:

It will be my responsibility to keep 'aronisstav/parallel' up to date with 'parapluu/master', whenever master advances. As soon as we start getting some benefit from the parallel version, I will start really merging 'aronisstav/parallel' into 'parapluu/master'. You should target 'aronisstav/parallel' in future pull requests (see it as 'your' master).

As a result, this pull request will not be merged. Please continue discussion in #37.