tripside/bullfinch — comment on commit 1c6f1bb32b98a23f36bad92c57791949fd19cd9f
File:  line 0
By tripside:

Hrm; there's an obstacle to doing so. The worker config includes a default process-by, but the minion never sees it; the config is encapsulated inside the worker. And the minion and worker are instantiated separately by the boss. Possible solutions i see:

* remove the ability for Workers to have individually configured defaults for process-by
* have the Minion get the default from the Worker every time (seems a little dirty; breaks encapsulation)
* have the Boss instantiate the Minion and the Minion instantiate the Worker (instead of the Boss instantiating both separately)

#2 is probably the easiest. #3 is probably the cleanest. Most all of the code inside the workerCount loop on line #226 could be moved inside the Minion. You could then remove the requirement that the kestrel client and the worker be passed to the constructor. Also, I'd make the connection to kestrel inside of the Minion thread rather than in the Boss. I don't know that that part is functionally different, but it seems cleaner.