Skip to content

Commit

Permalink
[BREAKING] Respect JOBS count if present
Browse files Browse the repository at this point in the history
For web-pack:

JOBS=0 uses the current thread of execution
JOBS=1..n spawns the corresponding thread or sub-processes
No set JOBS results in a dynamically calculated number of jobs

Previously

JOBS=1 used the current thread of execution
All other variations would dynamically calculated number of jobs
  • Loading branch information
stefanpenner committed Jul 21, 2021
1 parent fe30c4c commit 20292a2
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion packages/webpack/src/ember-webpack.ts
Original file line number Diff line number Diff line change
Expand Up @@ -522,6 +522,7 @@ const Webpack: PackagerConstructor<Options> = class Webpack implements Packager
};

const threadLoaderOptions = {
workers: 'JOBS' in process.env && Number(process.env.JOBS),
// poolTimeout shuts down idle workers. The problem is, for
// interactive rebuilds that means your startup cost for the
// next rebuild is at least 600ms worse. So we insist on
Expand All @@ -545,7 +546,7 @@ function warmUp(extraOptions: object | false | undefined) {
}

function maybeThreadLoader(isParallelSafe: boolean, extraOptions: object | false | undefined) {
if (process.env.JOBS === '1' || extraOptions === false || !isParallelSafe) {
if (process.env.JOBS === '0' || extraOptions === false || !isParallelSafe) {
return null;
}

Expand Down

0 comments on commit 20292a2

Please sign in to comment.