-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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.
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
Profiling Guide Requires Updates #447
Comments
@Trott Thank you for noticing this. I'll submit a PR with updates now that the tick processor is bundled with the node binary. |
The page is still referring to the v8 profiling tools in the source code (which I believe are no longer included). |
Good catch. Actually, the integration of the |
matthewloring
added a commit
that referenced
this issue
Mar 14, 2016
Instructions on using the tick processor now use the flag on the Node.js binary --prof-process for all versions and remove mention of processing scripts (which are now removed). Fixes #447
This issue was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In light of nodejs/node#3032, there is some stuff in the simple profiling guide that can be simplified. Maybe @matthewloring can indicate what needs to change? It might just be the part about making sure to use the same version of tick-processor as came with the source and how to fire up the tick processor? Not sure anything else...
The text was updated successfully, but these errors were encountered: