Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dont use workers for swc & esbuild minify #580

Open
ronakj opened this issue Oct 18, 2023 · 1 comment
Open

Dont use workers for swc & esbuild minify #580

ronakj opened this issue Oct 18, 2023 · 1 comment

Comments

@ronakj
Copy link

ronakj commented Oct 18, 2023

Modification Proposal

SWC and ESBuild minify both run off-thread though native bindings. We don't need to create workers to parallelize workloads for them, unlike terser or uglify-js which block the event loop if not run on separate worker. This should have performance improvement for people who have many output chunk files. I will try this out later and add some numbers to verify.

Expected Behavior / Situation

We simply use promise concurrenctly instead of workers to parallelize.

Actual Behavior / Situation

Workers are created regardless of minification implementation.

@alexander-akait
Copy link
Member

alexander-akait commented Dec 6, 2023

@ronakj Can you provide profiling? Because we use parallelization to compress multiple files and avoid memory problems with real big files

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants