vivekratnavel commented on pull request #921: URL: https://github.com/apache/hadoop-ozone/pull/921#issuecomment-648684073
I was curious and did a quick research on what other top Apache projects are following for their max line length limit and here is what I found: - Apache Spark (http://spark.apache.org/contributing.html) - 100 - Apache Kafka (https://kafka.apache.org/coding-guide) - certainly not 80 characters - Apache Hive (https://cwiki.apache.org/confluence/display/Hive/HowToContribute#HowToContribute-CodingConventions) - 120 - Apache Maven (http://maven.apache.org/developers/conventions/code.html) - 120 - Apache Ranger (https://cwiki.apache.org/confluence/display/RANGER/Coding+guidelines) - unlimited - Apache Lucene Solr (https://cwiki.apache.org/confluence/display/SOLR/HowToContribute#HowToContribute-ContributingCode(Features,BugFixes,Tests,etc...)) - 132 - Impala (https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=65868536) - 90 - Airflow (https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=99844429) - 110 - Apache cloudstack (https://cwiki.apache.org/confluence/display/CLOUDSTACK/Coding+conventions) - 180 - Apache IGNITE (https://cwiki.apache.org/confluence/display/IGNITE/Coding+Guidelines#CodingGuidelines-LineBreaking) - 120 - Apache Tomcat (https://github.com/apache/tomcat/blob/master/CONTRIBUTING.md#style-guide) - 100 - JMeter (https://cwiki.apache.org/confluence/display/JMETER/CodeStyleGuidelines) - soft 80, hard 120 - Apache ISIS (https://cwiki.apache.org/confluence/display/ISIS/Style+Guide) - 100 - Apache Traffic Server (https://cwiki.apache.org/confluence/display/TS/Coding+Style) - 132 - Apache Geode (https://cwiki.apache.org/confluence/display/GEODE/Code+Style+Guide) - 100 All of these projects seem to use >80 column limit for line lengths. I noticed that in many of these projects, even though they follow [Sun's conventions](http://web.archive.org/web/20140228225807/http://www.oracle.com/technetwork/java/codeconventions-150003.pdf), they specifically make an exception to the line length limit to make it >80 columns per line. ---------------------------------------------------------------- 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]
