Skip to content

Commit

Permalink
fix(js): replace deno_core with rquickjs (#1713)
Browse files Browse the repository at this point in the history
Co-authored-by: amit <[email protected]>
Co-authored-by: Tushar Mathur <[email protected]>
  • Loading branch information
3 people authored May 4, 2024
1 parent 51373fe commit 10b2797
Show file tree
Hide file tree
Showing 8 changed files with 790 additions and 570 deletions.
594 changes: 236 additions & 358 deletions Cargo.lock

Large diffs are not rendered by default.

8 changes: 3 additions & 5 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -96,9 +96,7 @@ lazy_static = "1.4.0"
which = { version = "6.0.1", optional = true }
async-recursion = "1.1.1"
tempfile = "3.10.1"
deno_core = { version = "0.278.0", optional = true, features = [
"v8_use_custom_libcxx",
], default-features = false }
rquickjs = { "version" = "0.5.1", optional = true, features = ["full"] }
strum_macros = "0.26.2"
# TODO: disable some levels with features?
tracing = "0.1.40"
Expand Down Expand Up @@ -167,7 +165,7 @@ tailcall-fixtures = { path = "./tailcall-fixtures" }

# Feature Flag to enable V8.
# V8 currently is not support on all platforms so, we control it via this feature flag.
js = ["dep:deno_core"]
js = ["dep:rquickjs"]

# Feature Flag to core CLI features.
# This is created to control what we expose for WASM.
Expand Down Expand Up @@ -233,4 +231,4 @@ harness = false

[[test]]
name = "execution_spec"
harness = false
harness = false
Loading

1 comment on commit 10b2797

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Running 30s test @ http://localhost:8000/graphql

4 threads and 100 connections

Thread Stats Avg Stdev Max +/- Stdev
Latency 7.49ms 3.56ms 102.29ms 74.14%
Req/Sec 3.39k 313.36 12.96k 97.42%

404578 requests in 30.10s, 2.03GB read

Requests/sec: 13441.29

Transfer/sec: 68.99MB

Please sign in to comment.