benchmark: inherit stdio/stderr instead of pipe

thisalihassan · marco-ippolito · commit 2274d0c8689c · 2024-05-03T08:30:30.000+02:00
PR-URL: #52456 Fixes: #52233 Refs: nodejs/performance#161 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Raz Luvaton <rluvaton@gmail.com>
diff --git a/benchmark/compare.js b/benchmark/compare.js
@@ -83,11 +83,8 @@ if (showProgress) {
     const spawnArgs = ['-c', cpuCore, cli.optional[job.binary], resolvedPath, ...cli.optional.set];
     child = spawn('taskset', spawnArgs, {
       env: process.env,
-      stdio: ['inherit', 'pipe', 'pipe', 'ipc'],
+      stdio: ['inherit', 'inherit', 'inherit', 'ipc'],
     });
-
-    child.stdout.pipe(process.stdout);
-    child.stderr.pipe(process.stderr);
   } else {
     child = fork(resolvedPath, cli.optional.set, {
       execPath: cli.optional[job.binary],
diff --git a/benchmark/run.js b/benchmark/run.js
@@ -54,11 +54,8 @@ if (format === 'csv') {
   let child;
   if (cpuCore !== null) {
     child = spawn('taskset', ['-c', cpuCore, 'node', scriptPath, ...args], {
-      stdio: ['inherit', 'pipe', 'pipe', 'ipc'],
+      stdio: ['inherit', 'inherit', 'inherit', 'ipc'],
     });
-
-    child.stdout.pipe(process.stdout);
-    child.stderr.pipe(process.stderr);
   } else {
     child = fork(
       scriptPath,
