tom-pytel commented on pull request #33: URL: https://github.com/apache/skywalking-nodejs/pull/33#issuecomment-787940653
> For such kind of operations (user local processing, callback or `.then()`), we can use Local Span (given proper operation name) to represent that local operations and it's acceptable to incur another Exit span in that local span. The thing is we do not know ahead of time what the user will do in the callback, so do we always pre-emptively create a local span on all Exit spans which call user code? This is essentially duplicating the Exit span everywhere and most of those Local spans will be unnecessary. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
