Skip to content

doc: note synchronous part of child_process.spawn #21234

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
davisjam wants to merge 4 commits into from
Closed

doc: note synchronous part of child_process.spawn #21234

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
d9fcde6
doc: note synchronous part of child_process.spawn
davisjam Jun 9, 2018
3119e9b
doc: add reference to V8 issue
davisjam Jun 10, 2018
217acc4
nit: UNIX-like OSes
davisjam Jun 15, 2018
bbfd361
nits: relocate and drop prefix 'Note:'
davisjam Jun 15, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
doc: add reference to V8 issue
  • Loading branch information
@davisjam
davisjam committed Jun 10, 2018
commit 3119e9bd037d705c7da74efec69be7e48171ab46
3 changes: 2 additions & 1 deletion doc/api/child_process.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,8 @@ the event loop until the spawned process either exits or is terminated.
*Note*: On some operating systems, the [`child_process.spawn()`][] method
performs memory operations synchronously before decoupling the event loop
Copy link
Copy Markdown
Member

@bnoordhuis bnoordhuis Jun 10, 2018

Choose a reason for hiding this comment

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

I'd be more specific and write "On UNIX-like operating systems" and tweak the wording to clarify that it's the underlying fork() system call that is responsible for copying memory, not child_process.spawn() (directly.)

Copy link
Copy Markdown
Contributor Author

@davisjam davisjam Jun 15, 2018 •
edited
Loading

Choose a reason for hiding this comment

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

I like "UNIX-like" (217acc4), but I don't think we need to go into detail on the root cause. The V8 link explains things.

from the child. Applications with a large memory footprint may find frequent
[`child_process.spawn()`][] calls to be a bottleneck.
[`child_process.spawn()`][] calls to be a bottleneck. For more information,
see [V8 issue 7381](https://bugs.chromium.org/p/v8/issues/detail?id=7381).

For convenience, the `child_process` module provides a handful of synchronous
and asynchronous alternatives to [`child_process.spawn()`][] and
Expand Down