[
https://issues.apache.org/jira/browse/HUDI-354?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
vinoyang updated HUDI-354:
--------------------------
Description:
This is an umbrella issue used to track apply some stricter comment and code
style validation rules for the whole project. The rules list below:
# All public classes must add class-level comments;
# All comments must end with a clear "."
# In the import statement of the class, clearly distinguish (by blank lines)
the import of Java SE and the import of non-java SE. Currently, I saw at least
two projects(Spark and Flink) that implement this rule. Flink implements
stricter rules than Spark. It is divided into several blocks from top to
bottom(owner import -> non-owner and non-JavaSE import -> Java SE import ->
static import), each block are sorted according to the natural sequence of
letters;
# Reconfirm the method and whether the comment is consistency;
Each project sub-module mappings to one subtask.
How to find all the invalidated points?
* Add the XML code snippet into {{PROJECT_ROOT/style/checkstyle.xml}} :
{code:java}
<module name="TreeWalker">
<!-- add the below modules-->
<!-- JAVADOC CHECKS -->
<module name="JavadocStyle">
<property name="severity" value="error"/>
</module>
<module name="JavadocType">
<property name="scope" value="protected"/>
<property name="severity" value="error"/>
<property name="allowMissingParamTags" value="true"/>
</module>
<module name="ImportOrder">
<!-- Checks for out of order import statements. -->
<property name="severity" value="error"/>
<property name="groups" value="org.apache.hudi,*,javax,java,scala"/>
<property name="separated" value="true"/>
<property name="sortStaticImportsAlphabetically" value="true"/>
<property name="option" value="bottom"/>
<property name="tokens" value="STATIC_IMPORT, IMPORT"/>
<message key="import.ordering"
value="Import {0} appears after other imports that it should precede"/>
</module>
<module name="RedundantImport">
<!-- Checks for redundant import statements. -->
<property name="severity" value="error"/>
<message key="import.redundancy"
value="Redundant import {0}."/>
</module>
<module name="AvoidStarImport">
<property name="severity" value="error"/>
</module>
</module>{code}
* Make sure you have installed CheckStyle-IDEA plugin and activated for the
project.
* Scan the project module you want to refactor and fix all the issues one by
one.
was:
This is an umbrella issue used to track apply some stricter comment and code
style validation rules for the whole project. The rules list below:
# All public classes must add class-level comments;
# All comments must end with a clear "."
# In the import statement of the class, clearly distinguish (by blank lines)
the import of Java SE and the import of non-java SE. Currently, I saw at least
two projects(Spark and Flink) that implement this rule. Flink implements
stricter rules than Spark. It is divided into several blocks from top to
bottom(owner import -> non-owner and non-JavaSE import -> Java SE import ->
static import), each block are sorted according to the natural sequence of
letters;
# Reconfirm the method and whether the comment is consistency;
Each project sub-module mappings to one subtask.
How to find all the invalidated points?
* Add the XML code snippet into {{PROJECT_ROOT/style/checkstyle.xml}} :
{code:java}
<module name="TreeWalker">
<!-- add the below modules-->
<!-- JAVADOC CHECKS -->
<module name="JavadocStyle">
<property name="severity" value="info"/>
</module>
<module name="JavadocType">
<property name="scope" value="protected"/>
<property name="severity" value="info"/>
<property name="allowMissingParamTags" value="true"/>
</module>
<module name="ImportOrder">
<!-- Checks for out of order import statements. -->
<property name="severity" value="info"/>
<property name="groups" value="org.apache.hudi,*,javax,java,scala"/>
<property name="separated" value="true"/>
<property name="sortStaticImportsAlphabetically" value="true"/>
<property name="option" value="bottom"/>
<property name="tokens" value="STATIC_IMPORT, IMPORT"/>
<message key="import.ordering"
value="Import {0} appears after other imports that it should precede"/>
</module>
<module name="RedundantImport">
<!-- Checks for redundant import statements. -->
<property name="severity" value="info"/>
<message key="import.redundancy"
value="Redundant import {0}."/>
</module>
<module name="AvoidStarImport">
<property name="severity" value="info"/>
</module>
</module>{code}
* Make sure you have installed CheckStyle-IDEA plugin and activated for the
project.
* Scan the project module you want to refactor and fix all the issues one by
one.
> Introduce stricter comment and code style validation rules
> ----------------------------------------------------------
>
> Key: HUDI-354
> URL: https://issues.apache.org/jira/browse/HUDI-354
> Project: Apache Hudi (incubating)
> Issue Type: Task
> Reporter: vinoyang
> Priority: Major
>
> This is an umbrella issue used to track apply some stricter comment and code
> style validation rules for the whole project. The rules list below:
> # All public classes must add class-level comments;
> # All comments must end with a clear "."
> # In the import statement of the class, clearly distinguish (by blank lines)
> the import of Java SE and the import of non-java SE. Currently, I saw at
> least two projects(Spark and Flink) that implement this rule. Flink
> implements stricter rules than Spark. It is divided into several blocks from
> top to bottom(owner import -> non-owner and non-JavaSE import -> Java SE
> import -> static import), each block are sorted according to the natural
> sequence of letters;
> # Reconfirm the method and whether the comment is consistency;
> Each project sub-module mappings to one subtask.
> How to find all the invalidated points?
> * Add the XML code snippet into {{PROJECT_ROOT/style/checkstyle.xml}} :
> {code:java}
> <module name="TreeWalker">
> <!-- add the below modules-->
> <!-- JAVADOC CHECKS -->
> <module name="JavadocStyle">
> <property name="severity" value="error"/>
> </module>
> <module name="JavadocType">
> <property name="scope" value="protected"/>
> <property name="severity" value="error"/>
> <property name="allowMissingParamTags" value="true"/>
> </module>
> <module name="ImportOrder">
> <!-- Checks for out of order import statements. -->
> <property name="severity" value="error"/>
> <property name="groups" value="org.apache.hudi,*,javax,java,scala"/>
> <property name="separated" value="true"/>
> <property name="sortStaticImportsAlphabetically" value="true"/>
> <property name="option" value="bottom"/>
> <property name="tokens" value="STATIC_IMPORT, IMPORT"/>
> <message key="import.ordering"
> value="Import {0} appears after other imports that it should precede"/>
> </module>
> <module name="RedundantImport">
> <!-- Checks for redundant import statements. -->
> <property name="severity" value="error"/>
> <message key="import.redundancy"
> value="Redundant import {0}."/>
> </module>
> <module name="AvoidStarImport">
> <property name="severity" value="error"/>
> </module>
> </module>{code}
> * Make sure you have installed CheckStyle-IDEA plugin and activated for the
> project.
> * Scan the project module you want to refactor and fix all the issues one by
> one.
>
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
