Skip to content

Commit

Permalink
Use non-default gzip options when compressing (#4729)
Browse files Browse the repository at this point in the history
* Use non-default gzip options when compressing

* Reduce non-determinism in tar

* use updated makefile for both builds

* try ustar format to discard pax headers

* remove time zone since it prints a warning

* add more options

* Revert changes

* Compress and measure only the wasm file for the QE

---------

Co-authored-by: Alexey Orlenko <alex@aqrln.net>
  • Loading branch information
Miguel Fernández and aqrln committed Feb 14, 2024
1 parent 69458ae commit ced8cab
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ build-qe-wasm:
build-qe-wasm-gz: build-qe-wasm
@cd query-engine/query-engine-wasm/pkg && \
for provider in postgresql mysql sqlite; do \
tar -zcvf $$provider.gz $$provider; \
gzip -knc $$provider/query_engine_bg.wasm > $$provider.gz; \
done;

build-schema-wasm:
Expand Down Expand Up @@ -372,7 +372,7 @@ test-driver-adapter-planetscale-wasm: test-planetscale-wasm
measure-qe-wasm: build-qe-wasm-gz
@cd query-engine/query-engine-wasm/pkg; \
for provider in postgresql mysql sqlite; do \
echo "$${provider}_size=$$(cat $$provider/* | wc -c | tr -d ' ')" >> $(ENGINE_SIZE_OUTPUT); \
echo "$${provider}_size=$$(cat $$provider/query_engine_bg.wasm | wc -c | tr -d ' ')" >> $(ENGINE_SIZE_OUTPUT); \
echo "$${provider}_size_gz=$$(cat $$provider.gz | wc -c | tr -d ' ')" >> $(ENGINE_SIZE_OUTPUT); \
done;

Expand Down

0 comments on commit ced8cab

Please sign in to comment.