[ https://issues.apache.org/jira/browse/KYLIN-3310?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Ted Yu updated KYLIN-3310: -------------------------- Description: lint helps identify structural problems. We should enable lint for maven-compiler-plugin {code} <artifactId>maven-compiler-plugin</artifactId> <version>${maven-compiler-plugin.version}</version> <configuration> <source>1.8</source> <target>1.8</target> <compilerArgs> <arg>-Xlint:all</arg> <arg>${compiler.error.flag}</arg> <!-- Override options warnings to support cross-compilation --> <arg>-Xlint:-options</arg> <!-- Temporary lint overrides, to be removed over time. --> <arg>-Xlint:-cast</arg> <arg>-Xlint:-deprecation</arg> <arg>-Xlint:-processing</arg> <arg>-Xlint:-rawtypes</arg> <arg>-Xlint:-serial</arg> <arg>-Xlint:-try</arg> <arg>-Xlint:-unchecked</arg> <arg>-Xlint:-varargs</arg> <!-- Uncomment the following args to display more warnings. --> <!-- -Xmaxwarns --> <!-- 10000 --> </compilerArgs> <showWarnings>true</showWarnings> <!-- Another temp override, to be set to true in due course. --> <showDeprecation>false</showDeprecation> </configuration> {code} was: lint helps identify structural problems. We should enable lint for maven-compiler-plugin {code} <artifactId>maven-compiler-plugin</artifactId> <version>${maven-compiler-plugin.version}</version> <configuration> <source>1.8</source> <target>1.8</target> <compilerArgs> <arg>-Xlint:all</arg> <arg>${compiler.error.flag}</arg> <!-- Override options warnings to support cross-compilation --> <arg>-Xlint:-options</arg> <!-- Temporary lint overrides, to be removed over time. --> <arg>-Xlint:-cast</arg> <arg>-Xlint:-deprecation</arg> <arg>-Xlint:-processing</arg> <arg>-Xlint:-rawtypes</arg> <arg>-Xlint:-serial</arg> <arg>-Xlint:-try</arg> <arg>-Xlint:-unchecked</arg> <arg>-Xlint:-varargs</arg> <!-- Uncomment the following args to display more warnings. --> <!-- -Xmaxwarns --> <!-- 10000 --> </compilerArgs> <showWarnings>true</showWarnings> <!-- Another temp override, to be set to true in due course. --> <showDeprecation>false</showDeprecation> </configuration> {code} > Use lint for maven-compiler-plugin > ---------------------------------- > > Key: KYLIN-3310 > URL: https://issues.apache.org/jira/browse/KYLIN-3310 > Project: Kylin > Issue Type: Improvement > Components: Tools, Build and Test > Reporter: Ted Yu > Priority: Major > > lint helps identify structural problems. > We should enable lint for maven-compiler-plugin > {code} > <artifactId>maven-compiler-plugin</artifactId> > <version>${maven-compiler-plugin.version}</version> > <configuration> > <source>1.8</source> > <target>1.8</target> > <compilerArgs> > <arg>-Xlint:all</arg> > <arg>${compiler.error.flag}</arg> > <!-- Override options warnings to support cross-compilation --> > <arg>-Xlint:-options</arg> > <!-- Temporary lint overrides, to be removed over time. --> > <arg>-Xlint:-cast</arg> > <arg>-Xlint:-deprecation</arg> > <arg>-Xlint:-processing</arg> > <arg>-Xlint:-rawtypes</arg> > <arg>-Xlint:-serial</arg> > <arg>-Xlint:-try</arg> > <arg>-Xlint:-unchecked</arg> > <arg>-Xlint:-varargs</arg> > <!-- Uncomment the following args to display more warnings. --> > <!-- -Xmaxwarns --> > <!-- 10000 --> > </compilerArgs> > <showWarnings>true</showWarnings> > <!-- Another temp override, to be set to true in due course. --> > <showDeprecation>false</showDeprecation> > </configuration> > {code} -- This message was sent by Atlassian JIRA (v7.6.3#76005)