yandrey321 commented on PR #10439: URL: https://github.com/apache/ozone/pull/10439#issuecomment-4650518120
> @yandrey321 , Why removing the debug messages? They can be enabled/disabled by setting the log level of BlockOutputStream. > > When there is a bugs, debug messages are very helpful. For the hot path the best practice is to limit amount of logging unless its placed for catching specific issue. JIT can do a lot of optimization and simplification/reducing amount of ifs on the hot path helps with JIT's execution path analysis as well as with low level optimizations. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
