VictorZeng commented on pull request #39: URL: https://github.com/apache/skywalking-java/pull/39#issuecomment-927505134
> > Apart from the comments inline, I'd also like to discuss whether we should also include part of the JSON string / bytes in the trace, for example, collect the first 64 bytes as a tag `content` and the `64` is configurable. > > Currently we only have the time cost in (de)serializing and the length of the content, but it makes little sense that we don't know what the content is when the time cost is too long, including part of the content might give the users some hints what content makes the JSON lib slower. > > Agree, collecting content is a risk. I prefer we don't do this. I think that providing JSON length can satisfy most user scenarios. It is not recommended to collect JSON content. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
