tomscut commented on pull request #3888: URL: https://github.com/apache/hadoop/pull/3888#issuecomment-1019931846
> @tomscut Sorry for being late. I agree with adding more EC logs, but I feel that the log information is too detailed, and some variables duplicate with other variables. Could you please make it more concise and descriptive? Thanks @tasanuma for your review. These states are related to each other, and if the upper and lower states don't match, there is something wrong with the process in between. Maybe `nonExcess` and `storage2index` are the same. But in some abnormal cases, we are not sure whether index will be calculated incorrectly, `storage2index` may cause elements of the same DN with different index to be overwritten. If simplify this log, I would remove `nonExcess` and keep the rest. Do you think this is OK? Thank you. -- 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]
