bshashikant edited a comment on pull request #1006: URL: https://github.com/apache/hadoop-ozone/pull/1006#issuecomment-638723740
> Thanks to open this issue @sadanand48. Unfortunately I don't have the context? (How did you notice this? What was the problem? How can I reproduce it?). > > Therefore, I have a few questions to understand this patch: > > Why don't we use `ERROR`, why do we need `INFO`? > > What is the expected number of log messages in case of any error? (Let's say disk is not writable). Is it possible to a lot of INFO message? > > The exception seems to be propagated. Is it logged on higher level? It should be logged in error level. It will be useful to log this error for debug purpose. The putBlock/watchForCommit exceptions are logged not the write chunk failures on higher level. If the disk is bad, its possible to have lot of error messages based on no of clients wriing on the disk. ---------------------------------------------------------------- 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]
