default value for numWorkers

CHANGELOG.md

@@ -9,6 +9,7 @@ v1.3.5 (14.02.2019)
  * **json**          - output as json
 - new console flag `w` to specify work dir
 - added ability to work without config file when at least one `overwrite` option has been specified
+- pool config now sets `numWorkers` equal to number of cores by default (this section can be omitted now)
 
 v1.3.4 (02.02.2019)
 -------------------

config.go

@@ -2,6 +2,7 @@ package roadrunner
 
 import (
 	"fmt"
+	"runtime"
 	"time"
 )
 
@@ -29,6 +30,7 @@ type Config struct {
 func (cfg *Config) InitDefaults() error {
 	cfg.AllocateTimeout = time.Minute
 	cfg.DestroyTimeout = time.Minute
+	cfg.NumWorkers = int64(runtime.NumCPU())
 
 	return nil
 }

config_test.go

@@ -17,6 +17,17 @@ func Test_NumWorkers(t *testing.T) {
 	assert.Equal(t, "pool.NumWorkers must be set", err.Error())
 }
 
+func Test_NumWorkers_Default(t *testing.T) {
+	cfg := Config{
+		AllocateTimeout: time.Second,
+		DestroyTimeout:  time.Second * 10,
+	}
+
+	assert.NoError(t, cfg.InitDefaults())
+	err := cfg.Valid()
+	assert.Nil(t, err)
+}
+
 func Test_AllocateTimeout(t *testing.T) {
 	cfg := Config{
 		NumWorkers:     10,