SWJTU-ZhangLei commented on PR #10376: URL: https://github.com/apache/doris/pull/10376#issuecomment-1168275914
> LGTM, but please fix the checks can you modify the checkstyle file ? [ERROR] /mnt/disk1/zhanglei/work/repo/incubator-doris/fe/fe-core/src/main/java/org/apache/doris/persist/EditLog.java:738: Line is longer than 120 characters (found 139). [LineLength] [ERROR] /mnt/disk1/zhanglei/work/repo/incubator-doris/fe/fe-core/src/test/java/org/apache/doris/ha/BDBHATest.java:17:14: Abbreviation in name 'BDBHATest' must contain no more than '5' consecutive capital letters. [AbbreviationAsWordInName] Audit done. [INFO] ------------------------------------------------------------------------ [INFO] Reactor Summary for fe-common 1.0-SNAPSHOT: [INFO] [INFO] fe-common .......................................... SUCCESS [ 26.103 s] [INFO] spark-dpp .......................................... SUCCESS [ 19.100 s] [INFO] fe-core ............................................ FAILURE [ 46.670 s] [INFO] ------------------------------------------------------------------------ [INFO] BUILD FAILURE [INFO] ------------------------------------------------------------------------ [INFO] Total time: 01:32 min [INFO] Finished at: 2022-06-28T14:15:35+08:00 [INFO] ------------------------------------------------------------------------ [ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:3.1.2:check (validate) on project fe-core: Failed during checkstyle execution: There are 2 errors reported by Checkstyle 9.3 with check/checkstyle/checkstyle.xml ruleset. -> [Help 1] -- 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]
