elek commented on pull request #1272: URL: https://github.com/apache/hadoop-ozone/pull/1272#issuecomment-697312216
> I think we should avoid logging chunk content. ContainerCommandRequestMessage implements related logic to clear data. I checked but it's not something which can be added easily. The `OzoneProtocolMessageDispatcher` is very generic. To support this one, it should be modified to accept a very specific function as an argument which transforms the original message just for the tracing. And would also introduce significant overhead in case of tracing is turned on. I am not sure if it's worth it, but I can add it if you think so. ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
