worker: use _writev in internal communication#33454
worker: use _writev in internal communication#33454addaleax wants to merge 1 commit intonodejs:masterfrom
Conversation
benjamingr
left a comment
There was a problem hiding this comment.
Changes LGTM.
The only concern I have here is whether or not we can somehow pass too much data here by postMessageing all the chunks at once. Also am a bit disappointed we don't have a niftier API for pushing multiple things at once (like Array.prototype.push has).
It’s only going to
Not sure, are you referring to |
|
CI: https://ci.nodejs.org/job/node-test-pull-request/31388/ (:yellow_heart:) |
PR-URL: #33454 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
|
Landed in d2a6f06 |
PR-URL: #33454 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
PR-URL: #33454 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Checklist
make -j4 test(UNIX), orvcbuild test(Windows) passes