Skip to content
  • Dongjoon Hyun's avatar
    7771c731
    [HOT-FIX][BUILD] Use the new location of `checkstyle-suppressions.xml` · 7771c731
    Dongjoon Hyun authored
    ## What changes were proposed in this pull request?
    
    This PR fixes `dev/lint-java` and `mvn checkstyle:check` failures due the recent file location change.
    The following is the error message of current master.
    ```
    Checkstyle checks failed at following occurrences:
    [ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:2.17:check (default-cli) on project spark-parent_2.11: Failed during checkstyle configuration: cannot initialize module SuppressionFilter - Cannot set property 'file' to 'checkstyle-suppressions.xml' in module SuppressionFilter: InvocationTargetException: Unable to find: checkstyle-suppressions.xml -> [Help 1]
    ```
    
    ## How was this patch tested?
    
    Manual. The following command should run correctly.
    ```
    ./dev/lint-java
    mvn checkstyle:check
    ```
    
    Author: Dongjoon Hyun <dongjoon@apache.org>
    
    Closes #11567 from dongjoon-hyun/hotfix_checkstyle_suppression.
    7771c731
    [HOT-FIX][BUILD] Use the new location of `checkstyle-suppressions.xml`
    Dongjoon Hyun authored
    ## What changes were proposed in this pull request?
    
    This PR fixes `dev/lint-java` and `mvn checkstyle:check` failures due the recent file location change.
    The following is the error message of current master.
    ```
    Checkstyle checks failed at following occurrences:
    [ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:2.17:check (default-cli) on project spark-parent_2.11: Failed during checkstyle configuration: cannot initialize module SuppressionFilter - Cannot set property 'file' to 'checkstyle-suppressions.xml' in module SuppressionFilter: InvocationTargetException: Unable to find: checkstyle-suppressions.xml -> [Help 1]
    ```
    
    ## How was this patch tested?
    
    Manual. The following command should run correctly.
    ```
    ./dev/lint-java
    mvn checkstyle:check
    ```
    
    Author: Dongjoon Hyun <dongjoon@apache.org>
    
    Closes #11567 from dongjoon-hyun/hotfix_checkstyle_suppression.
Loading