Skip to content

doc: improve the visibility of Worker threads cli options #31380

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
HarshithaKP wants to merge 4 commits into from
Closed

doc: improve the visibility of Worker threads cli options #31380

Changes from 3 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
fbebd40
doc: visibility of Worker threads cli options
HarshithaKP Jan 16, 2020
c443fab
fixup: address review comments
HarshithaKP Jan 23, 2020
086b992
fixup: address review comment
HarshithaKP Jan 23, 2020
7e871d1
fixup: address review comment
HarshithaKP Feb 12, 2020
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
5 changes: 5 additions & 0 deletions doc/api/worker_threads.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,10 @@ The above example spawns a Worker thread for each `parse()` call. In actual
practice, use a pool of Workers instead for these kinds of tasks. Otherwise, the
overhead of creating Workers would likely exceed their benefit.

Worker threads inherit non-process-specific options by default. Refer to
[`Worker constructor options`][] to know how to customize worker thread options,
specifically `argv` and `execArgv` options.
Copy link
Copy Markdown
Member

@addaleax addaleax Feb 4, 2020

Choose a reason for hiding this comment

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

The two paragraphs here belong together; this should be added either above, or (preferably) below them.

Copy link
Copy Markdown
Member Author

@HarshithaKP HarshithaKP Feb 12, 2020

Choose a reason for hiding this comment

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

@addaleax, Thanks. Put it in the suggested place.


When implementing a worker pool, use the [`AsyncResource`][] API to inform
diagnostic tools (e.g. in order to provide asynchronous stack traces) about the
correlation between tasks and their outcomes.
Expand Down Expand Up @@ -744,6 +748,7 @@ active handle in the event system. If the worker is already `unref()`ed calling
[`require('worker_threads').workerData`]: #worker_threads_worker_workerdata
[`trace_events`]: tracing.html
[`vm`]: vm.html
[`Worker constructor options`]: #worker_threads_new_worker_filename_options
[`worker.on('message')`]: #worker_threads_event_message_1
[`worker.postMessage()`]: #worker_threads_worker_postmessage_value_transferlist
[`worker.SHARE_ENV`]: #worker_threads_worker_share_env
Expand Down