Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit fbce317

Browse filesBrowse files
AdamMajertargos
authored andcommitted
test: reduce memory usage of test-worker-stdio
On systems with limited memory and that are compiled with debugging information, this particular test is causing OOM condition especially as it is run in parallel. Even when run with a stripped binary as an input, the test consumes upward of 250M RSS. By limiting the input stream to the 1M, the stream is still buffering but memory consumption is similar to other parallel tests. PR-URL: #37769 Reviewed-By: Michael Dawson <midawson@redhat.com>
1 parent 619ac79 commit fbce317
Copy full SHA for fbce317

File tree

Expand file treeCollapse file tree

1 file changed

+1
-1
lines changed
Open diff view settings
Filter options
Expand file treeCollapse file tree

1 file changed

+1
-1
lines changed
Open diff view settings
Collapse file

‎test/parallel/test-worker-stdio.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-worker-stdio.js
+1-1Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ if (isMainThread) {
2727
const passed = new BufferingWritable();
2828

2929
const w = new Worker(__filename, { stdin: true, stdout: true });
30-
const source = fs.createReadStream(process.execPath);
30+
const source = fs.createReadStream(process.execPath, { end: 1_000_000 });
3131
source.pipe(w.stdin);
3232
source.pipe(original);
3333
w.stdout.pipe(passed);

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.