database/doris
2
0
Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

[style](fe)the last step of fe CheckStyle (#10134)

Browse Source 
1. fix all checkstyle warning
2. change all checkstyle rules to error
3. remove some java doc rules
    a. RequireEmptyLineBeforeBlockTagGroup
    b. JavadocStyle
    c. JavadocParagraph
4. suppress some rules for old codes
    a. all java doc rules only affect on Nereids
    b. DeclarationOrder only affect on Nereids
    c. OverloadMethodsDeclarationOrder only affect on Nereids
    d. VariableDeclarationUsageDistance only affect on Nereids
    e. suppress OneTopLevelClass on org/apache/doris/load/loadv2/dpp/ColumnParser.java
    f. suppress OneTopLevelClass on org/apache/doris/load/loadv2/dpp/SparkRDDAggregator.java
    g. suppress LineLength on org/apache/doris/catalog/FunctionSet.java
    h. suppress LineLength on org/apache/doris/common/ErrorCode.java
This commit is contained in:
morrySnow
2022-06-17 21:02:45 +08:00
committed by GitHub
parent fea815f290
commit b7b78ae707
514 changed files with 3877 additions and 2757 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
docs/en/developer/developer-guide/java-format-code.md
View File

@ -42,6 +42,13 @@ standard java package
* Do not use `import *`
* Do not use `import static`
## Check when compile
Now, when compiling with `caven`, `CheckStyle` checks are done by default. This will slightly slow down compilation. If you want to skip checkstyle, please use the following command to compile
```
mvn clean install -DskipTests -Dcheckstyle.skip
```
## Checkstyle Plugin
Now we have `formatter-check` in `CI` to check the code format.