sodonnel commented on PR #921: URL: https://github.com/apache/ozone/pull/921#issuecomment-1842489014
> Could anyone who has a binding vote confirm my challenge? I can confirm the challenge. I also agree, the cited article is about written text, and reading source code is not the same as reading natural language text. In general natural language is scanned and read very quickly, so long lines make this difficult. Code, generally gets scanned slowly as one has to take time to understand it. The extra line breaks and indentations can make it harder to follow. I do agree we need a line length to avoid very long lines, and 120 seems to be a limit many projects have adopted. Even Hadoop, which this project forked from moved from 80 to 120 lines a year or more back, with no issues I have encountered. -- 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]
