diff options
| author | Bartek Iwańczuk <biwanczuk@gmail.com> | 2019-11-17 14:14:50 +0100 |
|---|---|---|
| committer | Ry Dahl <ry@tinyclouds.org> | 2019-11-17 08:14:50 -0500 |
| commit | f356b2bd5e3d46eaf4147a38a2f7c7e7c2824fbf (patch) | |
| tree | 3a06e7c15fcbba0587c498b11c498d4b9494c11c /cli/compilers/wasm.rs | |
| parent | 2b3afda625c5818f01a45f52075ac04307224b30 (diff) | |
refactor: fixes for futures (#3363)
After landing #3358 the benchmarks exploded indicating problems with workers and deno_core_http_bench.
This PR dramatically fixes thread/syscall count that showed up on benchmarks. Thread count is not back to previous levels but difference went from hundreds/thousands to about ~50.
Diffstat (limited to 'cli/compilers/wasm.rs')
| -rw-r--r-- | cli/compilers/wasm.rs | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/cli/compilers/wasm.rs b/cli/compilers/wasm.rs index 30a171db4..882e28e43 100644 --- a/cli/compilers/wasm.rs +++ b/cli/compilers/wasm.rs @@ -86,14 +86,13 @@ impl WasmCompiler { let worker_ = worker.clone(); let url = source_file.url.clone(); + let _res = worker.post_message( + serde_json::to_string(&base64_data) + .unwrap() + .into_boxed_str() + .into_boxed_bytes(), + ); let fut = worker - .post_message( - serde_json::to_string(&base64_data) - .unwrap() - .into_boxed_str() - .into_boxed_bytes(), - ) - .then(move |_| worker) .then(move |result| { if let Err(err) = result { // TODO(ry) Need to forward the error instead of exiting. |