Skip to content
Snippets Groups Projects
  • Xianyang Liu's avatar
    fcb88f92
    [MINOR][BUILD] Fix lint-java breaks. · fcb88f92
    Xianyang Liu authored
    ## What changes were proposed in this pull request?
    
    This PR proposes to fix the lint-breaks as below:
    ```
    [ERROR] src/main/java/org/apache/spark/unsafe/Platform.java:[51] (regexp) RegexpSingleline: No trailing whitespace allowed.
    [ERROR] src/main/scala/org/apache/spark/sql/streaming/Trigger.java:[45,25] (naming) MethodName: Method name 'ProcessingTime' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9_]*$'.
    [ERROR] src/main/scala/org/apache/spark/sql/streaming/Trigger.java:[62,25] (naming) MethodName: Method name 'ProcessingTime' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9_]*$'.
    [ERROR] src/main/scala/org/apache/spark/sql/streaming/Trigger.java:[78,25] (naming) MethodName: Method name 'ProcessingTime' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9_]*$'.
    [ERROR] src/main/scala/org/apache/spark/sql/streaming/Trigger.java:[92,25] (naming) MethodName: Method name 'ProcessingTime' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9_]*$'.
    [ERROR] src/main/scala/org/apache/spark/sql/streaming/Trigger.java:[102,25] (naming) MethodName: Method name 'Once' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9_]*$'.
    [ERROR] src/test/java/org/apache/spark/streaming/kinesis/JavaKinesisInputDStreamBuilderSuite.java:[28,8] (imports) UnusedImports: Unused import - org.apache.spark.streaming.api.java.JavaDStream.
    ```
    
    after:
    ```
    dev/lint-java
    Checkstyle checks passed.
    ```
    [Test Result](https://travis-ci.org/ConeyLiu/spark/jobs/229666169)
    
    ## How was this patch tested?
    
    Travis CI
    
    Author: Xianyang Liu <xianyang.liu@intel.com>
    
    Closes #17890 from ConeyLiu/codestyle.
    fcb88f92
    History
    [MINOR][BUILD] Fix lint-java breaks.
    Xianyang Liu authored
    ## What changes were proposed in this pull request?
    
    This PR proposes to fix the lint-breaks as below:
    ```
    [ERROR] src/main/java/org/apache/spark/unsafe/Platform.java:[51] (regexp) RegexpSingleline: No trailing whitespace allowed.
    [ERROR] src/main/scala/org/apache/spark/sql/streaming/Trigger.java:[45,25] (naming) MethodName: Method name 'ProcessingTime' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9_]*$'.
    [ERROR] src/main/scala/org/apache/spark/sql/streaming/Trigger.java:[62,25] (naming) MethodName: Method name 'ProcessingTime' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9_]*$'.
    [ERROR] src/main/scala/org/apache/spark/sql/streaming/Trigger.java:[78,25] (naming) MethodName: Method name 'ProcessingTime' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9_]*$'.
    [ERROR] src/main/scala/org/apache/spark/sql/streaming/Trigger.java:[92,25] (naming) MethodName: Method name 'ProcessingTime' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9_]*$'.
    [ERROR] src/main/scala/org/apache/spark/sql/streaming/Trigger.java:[102,25] (naming) MethodName: Method name 'Once' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9_]*$'.
    [ERROR] src/test/java/org/apache/spark/streaming/kinesis/JavaKinesisInputDStreamBuilderSuite.java:[28,8] (imports) UnusedImports: Unused import - org.apache.spark.streaming.api.java.JavaDStream.
    ```
    
    after:
    ```
    dev/lint-java
    Checkstyle checks passed.
    ```
    [Test Result](https://travis-ci.org/ConeyLiu/spark/jobs/229666169)
    
    ## How was this patch tested?
    
    Travis CI
    
    Author: Xianyang Liu <xianyang.liu@intel.com>
    
    Closes #17890 from ConeyLiu/codestyle.
checkstyle-suppressions.xml 2.35 KiB
<!--
  ~ Licensed to the Apache Software Foundation (ASF) under one or more
  ~ contributor license agreements.  See the NOTICE file distributed with
  ~ this work for additional information regarding copyright ownership.
  ~ The ASF licenses this file to You under the Apache License, Version 2.0
  ~ (the "License"); you may not use this file except in compliance with
  ~ the License.  You may obtain a copy of the License at
  ~
  ~    http://www.apache.org/licenses/LICENSE-2.0
  ~
  ~ Unless required by applicable law or agreed to in writing, software
  ~ distributed under the License is distributed on an "AS IS" BASIS,
  ~ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
  ~ See the License for the specific language governing permissions and
  ~ limitations under the License.
  -->

<!DOCTYPE suppressions PUBLIC
"-//Puppy Crawl//DTD Suppressions 1.1//EN"
"http://www.puppycrawl.com/dtds/suppressions_1_1.dtd">

<!--

    This file contains suppression rules for Checkstyle checks.
    Ideally only files that cannot be modified (e.g. third-party code)
    should be added here. All other violations should be fixed.

-->

<suppressions>
    <suppress checks=".*"
              files="core/src/main/java/org/apache/spark/util/collection/TimSort.java"/>
    <suppress checks=".*"
              files="sql/core/src/main/java/org/apache/spark/sql/api.java/*"/>
    <suppress checks="LineLength"
              files="src/test/java/org/apache/spark/sql/hive/test/Complex.java"/>
    <suppress checks="LineLength"
              files="src/main/java/org/apache/spark/examples/JavaLogQuery.java"/>
    <suppress checks="LineLength"
              files="src/main/java/org/apache/hive/service/*"/>
    <suppress checks="MethodName"
              files="src/main/java/org/apache/hive/service/auth/PasswdAuthenticationProvider.java"/>
    <suppress checks="NoFinalizer"
              files="src/main/java/org/apache/hive/service/server/ThreadWithGarbageCleanup.java"/>
    <suppress checks="MethodName"
              files="sql/catalyst/src/main/java/org/apache/spark/sql/streaming/OutputMode.java"/>
    <suppress checks="MethodName"
              files="sql/catalyst/src/main/java/org/apache/spark/sql/streaming/GroupStateTimeout.java"/>
    <suppress checks="MethodName"
              files="sql/core/src/main/java/org/apache/spark/sql/streaming/Trigger.java"/>
</suppressions>