Skip to content

Commit fcb88f9

Browse files
ConeyLiusrowen
authored andcommitted
[MINOR][BUILD] Fix lint-java breaks.
## 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 <[email protected]> Closes #17890 from ConeyLiu/codestyle.
1 parent 76e4a55 commit fcb88f9

File tree

4 files changed

+2
-6
lines changed

4 files changed

+2
-6
lines changed

common/unsafe/src/main/java/org/apache/spark/unsafe/Platform.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ public final class Platform {
4848
boolean _unaligned;
4949
String arch = System.getProperty("os.arch", "");
5050
if (arch.equals("ppc64le") || arch.equals("ppc64")) {
51-
// Since java.nio.Bits.unaligned() doesn't return true on ppc (See JDK-8165231), but
51+
// Since java.nio.Bits.unaligned() doesn't return true on ppc (See JDK-8165231), but
5252
// ppc64 and ppc64le support it
5353
_unaligned = true;
5454
} else {

core/src/main/scala/org/apache/spark/storage/BlockManager.scala

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -29,8 +29,6 @@ import scala.reflect.ClassTag
2929
import scala.util.Random
3030
import scala.util.control.NonFatal
3131

32-
import com.google.common.io.ByteStreams
33-
3432
import org.apache.spark._
3533
import org.apache.spark.executor.{DataReadMethod, ShuffleWriteMetrics}
3634
import org.apache.spark.internal.Logging
@@ -41,7 +39,6 @@ import org.apache.spark.network.netty.SparkTransportConf
4139
import org.apache.spark.network.shuffle.ExternalShuffleClient
4240
import org.apache.spark.network.shuffle.protocol.ExecutorShuffleInfo
4341
import org.apache.spark.rpc.RpcEnv
44-
import org.apache.spark.security.CryptoStreamUtils
4542
import org.apache.spark.serializer.{SerializerInstance, SerializerManager}
4643
import org.apache.spark.shuffle.ShuffleManager
4744
import org.apache.spark.storage.memory._

dev/checkstyle-suppressions.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,5 +47,5 @@
4747
<suppress checks="MethodName"
4848
files="sql/catalyst/src/main/java/org/apache/spark/sql/streaming/GroupStateTimeout.java"/>
4949
<suppress checks="MethodName"
50-
files="sql/catalyst/src/main/java/org/apache/spark/sql/streaming/Trigger.java"/>
50+
files="sql/core/src/main/java/org/apache/spark/sql/streaming/Trigger.java"/>
5151
</suppressions>

external/kinesis-asl/src/test/java/org/apache/spark/streaming/kinesis/JavaKinesisInputDStreamBuilderSuite.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@
2525
import org.apache.spark.streaming.Duration;
2626
import org.apache.spark.streaming.Seconds;
2727
import org.apache.spark.streaming.LocalJavaStreamingContext;
28-
import org.apache.spark.streaming.api.java.JavaDStream;
2928

3029
public class JavaKinesisInputDStreamBuilderSuite extends LocalJavaStreamingContext {
3130
/**

0 commit comments

Comments
 (0)