Skip to content

Commit

Permalink
[feat] Improve performance by compressing in parallel (#6710)
Browse files Browse the repository at this point in the history
* sanjaiyan: Improve performance by running functions in parallel

* Update index.js

* Update index.js

* format

* remove duplicate line

* conver to Promise.all

* Create twenty-apricots-grin.md

Co-authored-by: Ben McCann <[email protected]>
  • Loading branch information
sanjaiyan-dev and benmccann authored Sep 12, 2022
1 parent 40e5647 commit ec88a70
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 8 deletions.
6 changes: 6 additions & 0 deletions .changeset/twenty-apricots-grin.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
---
"@sveltejs/adapter-node": patch
"@sveltejs/adapter-static": patch
---

Improve performance by compressing in parallel
6 changes: 4 additions & 2 deletions packages/adapter-node/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,10 @@ export default function (opts = {}) {

if (precompress) {
builder.log.minor('Compressing assets');
await builder.compress(`${out}/client`);
await builder.compress(`${out}/prerendered`);
await Promise.all([
builder.compress(`${out}/client`),
builder.compress(`${out}/prerendered`)
]);
}

builder.log.minor('Building server');
Expand Down
8 changes: 2 additions & 6 deletions packages/adapter-static/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -64,15 +64,11 @@ export default function (options) {
builder.writePrerendered(pages, { fallback });

if (precompress) {
builder.log.minor('Compressing assets and pages');
if (pages === assets) {
builder.log.minor('Compressing assets and pages');
await builder.compress(assets);
} else {
builder.log.minor('Compressing assets');
await builder.compress(assets);

builder.log.minor('Compressing pages');
await builder.compress(pages);
await Promise.all([builder.compress(assets), builder.compress(pages)]);
}
}

Expand Down

0 comments on commit ec88a70

Please sign in to comment.