-
Notifications
You must be signed in to change notification settings - Fork 404
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
"cpu time" vs "request time" #122
Comments
This could be tested with var http = require('http');
http.createServer(function (req, res) {
setTimeout(function(){
res.writeHead(200, {'Content-Type': 'text/plain'});
res.end('Hello World\n');
}, 5000)
}).listen(1337, '127.0.0.1');
console.log('Server running at http://127.0.0.1:1337/'); "real time" would always be 5s, and cpu time would be something like 10ms for each request |
There's a couple obstacles to doing this in a way that's not misleading:
It is an ongoing project of ours to improve New Relic for Node's reporting of parent-time vs child-time function execution in transaction traces (which will bring with it improvements to the transaction breakdown charts), which is where getting better at the first item on the list above will yield significant benefits, but due to how all the data is represented (and the representation's inbuilt assumptions that the timings are being generated from synchronous processes), it's a little harder than it looks. Good idea, though! Thanks! |
Thanks for all the information! Good luck! |
There is nothing actionable for the Node.js agent at this point, so I'm closing this issue. Thanks for the feedback on a requested feature! |
…/follow-redirects-1.15.6 build(deps-dev): bump follow-redirects from 1.15.4 to 1.15.6
update actions that are using deprecated node 12
Fixes nr-hooks version checking circular reference issue.
Fixes nr-hooks version checking circular reference issue.
I'm curious if there is a way to instrument a request which will report on both the "real request time" (time between request and response) vs (CPU time per request).
NewRelic is clear to say
by percent of wall clock time
, which means "request time", but it would be useful to be able to generically see the "load" each request takes.The text was updated successfully, but these errors were encountered: