dmsolr commented on pull request #42: URL: https://github.com/apache/skywalking-nginx-lua/pull/42#issuecomment-699574290
In my mind, the Span ERROR means Nginx has some wrong or upstream returns an unsuccess(include 4xx and 5xx). I think that will make confusion to others. Now, Tag is searchable. So, I think to make a new Tag to represent client_error(4xx), server_error(5xx). That will be better. ---------------------------------------------------------------- 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]
