wix-incubator / lerna-script

Lerna addon for adding custom tasks
MIT License
164 stars 13 forks source link

limit the number of concurrent npm processes and increase prompt page size #396

Closed eyalpost closed 4 years ago

hugebdu commented 4 years ago

@eyalpost why needed? why 8?

eyalpost commented 4 years ago

when running lerna dependencies:latest script, my mac hangs due to the large number of processes that are created by this script (basically it runs npm info for each module). This bounds the number of processes created (8 seems the best without significantly affecting performance)

hugebdu commented 4 years ago

and what's the current limit? is it some math of # of processors?

maybe let's make it configurable, WDYT?

eyalpost commented 4 years ago

currently there is no limit. if there are 200 modules to check then 200 processes will be created. Why make it configurable? Who will configure it?

hugebdu commented 4 years ago

Why make it configurable? Who will configure it?

ok, but let's make it number of cores available. IMHO more reasonable then just 8