wu-sheng commented on pull request #7070: URL: https://github.com/apache/skywalking/pull/7070#issuecomment-855171114
> LGTM. If @JaredTan95 you have time, please also consider https://github.com/apache/skywalking/pull/7070#discussion_r645927919 My point is, as segment has been ignored, segment is not meaningful. So outputing trace id may be better. If we want a polish, it is better to have an endpoint name or operation name of first span in log, should be more helpful. -- 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]
