[ 
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)

Reply via email to