perf(runtime/spawn): collect output using op_read_all

[marcosc90]

This patch

benchmark      time (avg)             (min … max)       p75       p99      p995
------------------------------------------------- -----------------------------
echo deno   23.99 ms/iter   (22.51 ms … 33.61 ms)  23.97 ms  33.61 ms  33.61 ms
cat 16kb    24.27 ms/iter    (22.5 ms … 35.21 ms)   24.2 ms  35.21 ms  35.21 ms
cat 1mb     25.88 ms/iter   (25.04 ms … 30.28 ms)  26.12 ms  30.28 ms  30.28 ms
cat 15mb    38.41 ms/iter       (35.7 ms … 50 ms)  38.31 ms     50 ms     50 ms

main

benchmark      time (avg)             (min … max)       p75       p99      p995
------------------------------------------------- -----------------------------
echo deno   35.66 ms/iter   (34.53 ms … 41.84 ms)  35.79 ms  41.84 ms  41.84 ms
cat 16kb    35.99 ms/iter   (34.52 ms … 44.94 ms)  36.05 ms  44.94 ms  44.94 ms
cat 1mb     38.68 ms/iter   (36.67 ms … 50.44 ms)  37.95 ms  50.44 ms  50.44 ms
cat 15mb     48.4 ms/iter   (46.19 ms … 58.41 ms)  49.16 ms  58.41 ms  58.41 ms

[littledivy]

@marcosc90 Can you drop the benchmark source somewhere in cli/bench?

[littledivy]

I think LGTM! Thanks.

[littledivy]

@lucacasonato can you take a look? 🙏

[marcosc90]

@marcosc90 Can you drop the benchmark source somewhere in cli/bench?

Done @littledivy

[marcosc90]

command_test.ts tests are flaky (or broken) in main

https://github.com/denoland/deno/actions/runs/3457193770/jobs/5770460515#step:34:3397

[lucacasonato]

Some minor comments. Looks great overall.

[lucacasonato]

Suggested change

	function readableStreamUnrefable(stream) {
	function readableStreamIsUnrefable(stream) {

[lucacasonato]

This distinction exists to prevent unrefable streams being used in regular fast streams that are unaware of refability? Please add a comment

[marcosc90]

Correct, added the comment.

[bartlomieju]

LGTM, thank you @marcosc90!