FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

benchmark: fix worker startup benchmark by joyeecheung · Pull Request #46680 · nodejs/node · GitHub

/ node Public

benchmark: fix worker startup benchmark - #46680

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
joyeecheung:fix-startup-bench
Feb 23, 2023
Merged

benchmark: fix worker startup benchmark#46680
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
joyeecheung:fix-startup-bench

Conversation

Copy link
Copy Markdown
Member

It previously called spawnProcess in spawnWorker, which was incorrect.

It previously called spawnProcess in spawnWorker, which was
incorrect.

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/startup

nodejs-github-bot added the benchmark Issues and PRs related to the benchmark subsystem. label Feb 16, 2023
anonrig added the performance Issues and PRs related to the performance of Node.js. label Feb 20, 2023

Copy link
Copy Markdown
Collaborator

joyeecheung added the commit-queue Add this label to land a pull request using GitHub Actions. label Feb 23, 2023
nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Feb 23, 2023
nodejs-github-bot merged commit 9e06ef0 into nodejs:main Feb 23, 2023

Copy link
Copy Markdown
Collaborator

Landed in 9e06ef0

targos pushed a commit that referenced this pull request Mar 13, 2023
It previously called spawnProcess in spawnWorker, which was
incorrect.

PR-URL: #46680
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
danielleadams pushed a commit that referenced this pull request Apr 11, 2023
It previously called spawnProcess in spawnWorker, which was
incorrect.

PR-URL: #46680
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

benchmark Issues and PRs related to the benchmark subsystem. performance Issues and PRs related to the performance of Node.js.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants


Back | FazBrowse Home | New Git URL