Stamatis Zampetakis created HIVE-29182:
------------------------------------------

             Summary: Upgrade maven-checkstyle-plugin version from 2.17 to 3.5.0
                 Key: HIVE-29182
                 URL: https://issues.apache.org/jira/browse/HIVE-29182
             Project: Hive
          Issue Type: Bug
            Reporter: Stamatis Zampetakis
            Assignee: Stamatis Zampetakis


The majority of the modules (except Iceberg) use a pretty old version of the 
plugin. The 2.17 was [released on 
2015-10-15|https://maven.apache.org/plugins/maven-checkstyle-plugin/history.html]
 and by default it uses Checkstyle 6.11.2, which is also very old.

The 2.17/6.11.2 version has various bugs that prevent the checks from running 
in many modules. Currently the following command fails on master: (commit 
97a9677f6ca5561eceab0b6bb5fa14b6c9818803).
{noformat}
$ mvn clean install -DskipTests -Pitests
$ mvn checkstyle:checkstyle -Pitests -fn
...
[ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-checkstyle-plugin:2.17:checkstyle (default-cli) 
on project hive-common: An error has occurred in Checkstyle report generation.: 
Failed during checkstyle configuration: TokenStreamRecognitionException 
occurred during the analysis of file 
/home/zabetak/Projects/Apache/hive/common/src/java/org/apache/hadoop/hive/common/format/datetime/HiveSqlDateTimeFormatter.java.
 unexpected char: 0xA -> [Help 1]
[ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-checkstyle-plugin:2.17:checkstyle (default-cli) 
on project hive-standalone-metastore-client: An error has occurred in 
Checkstyle report generation.: Failed during checkstyle execution: Unable to 
find configuration file at location: 
/home/zabetak/Projects/Apache/hive/standalone-metastore/metastore-client/./checkstyle/checkstyle.xml:
 Could not find resource 
'/home/zabetak/Projects/Apache/hive/standalone-metastore/metastore-client/./checkstyle/checkstyle.xml'.
 -> [Help 1]
[ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-checkstyle-plugin:2.17:checkstyle (default-cli) 
on project hive-standalone-metastore-server: An error has occurred in 
Checkstyle report generation.: Failed during checkstyle configuration: 
MismatchedTokenException occurred during the analysis of file 
/home/zabetak/Projects/Apache/hive/standalone-metastore/metastore-server/src/test/java/org/apache/hadoop/hive/metastore/dbinstall/rules/MetastoreRuleFactory.java.
 expecting EOF, found 'case' -> [Help 1]
[ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-checkstyle-plugin:2.17:checkstyle (default-cli) 
on project hive-exec: An error has occurred in Checkstyle report generation.: 
Failed during checkstyle configuration: MismatchedTokenException occurred 
during the analysis of file 
/home/zabetak/Projects/Apache/hive/ql/src/java/org/apache/hadoop/hive/ql/io/CombineHiveInputFormat.java.
 expecting EOF, found '}' -> [Help 1]
[ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-checkstyle-plugin:2.17:checkstyle (default-cli) 
on project hive-hcatalog-core: An error has occurred in Checkstyle report 
generation.: Failed during checkstyle configuration: MismatchedTokenException 
occurred during the analysis of file 
/home/zabetak/Projects/Apache/hive/hcatalog/core/src/main/java/org/apache/hive/hcatalog/common/HCatUtil.java.
 expecting EOF, found 'case' -> [Help 1]
{noformat}
Upgrading to 3.5.0 resolves most of the errors above and allows us to take 
advantage all the recent improvements done in the plugin and Checkstyle. The 
3.5.0 version is also the one used currently in the Iceberg modules so it makes 
the entire handling more uniform.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to