From 045fa87a3db5d9954d1ca88c8a2ef28c11214330 Mon Sep 17 00:00:00 2001
From: lamber-ken <xl38154@qq.com>
Date: Thu, 14 Nov 2019 02:39:05 +0800
Subject: [PATCH] [HUDI-330] add EmptyStatement java checkstyle rule

---
 hudi-client/src/test/java/org/apache/hudi/TestCleaner.java | 6 +++---
 style/checkstyle.xml                                       | 2 ++
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/hudi-client/src/test/java/org/apache/hudi/TestCleaner.java b/hudi-client/src/test/java/org/apache/hudi/TestCleaner.java
index 293ddd04c..021f13d00 100644
--- a/hudi-client/src/test/java/org/apache/hudi/TestCleaner.java
+++ b/hudi-client/src/test/java/org/apache/hudi/TestCleaner.java
@@ -499,7 +499,7 @@ public class TestCleaner extends TestHoodieClientBase {
     HoodieTestUtils.createDataFile(basePath, HoodieTestDataGenerator.DEFAULT_FIRST_PARTITION_PATH, "001", file1P0C0); // update
     HoodieTestUtils.createDataFile(basePath, HoodieTestDataGenerator.DEFAULT_SECOND_PARTITION_PATH, "001", file1P1C0); // update
 
-    List<HoodieCleanStat> hoodieCleanStatsTwo = runCleaner(config);;
+    List<HoodieCleanStat> hoodieCleanStatsTwo = runCleaner(config);
     assertEquals("Must clean 1 file", 1,
         getCleanStat(hoodieCleanStatsTwo, HoodieTestDataGenerator.DEFAULT_FIRST_PARTITION_PATH).getSuccessDeleteFiles()
             .size());
@@ -525,7 +525,7 @@ public class TestCleaner extends TestHoodieClientBase {
     String file3P0C2 =
         HoodieTestUtils.createNewDataFile(basePath, HoodieTestDataGenerator.DEFAULT_FIRST_PARTITION_PATH, "002");
 
-    List<HoodieCleanStat> hoodieCleanStatsThree = runCleaner(config);;
+    List<HoodieCleanStat> hoodieCleanStatsThree = runCleaner(config);
     assertEquals("Must clean two files", 2,
         getCleanStat(hoodieCleanStatsThree, HoodieTestDataGenerator.DEFAULT_FIRST_PARTITION_PATH)
             .getSuccessDeleteFiles().size());
@@ -583,7 +583,7 @@ public class TestCleaner extends TestHoodieClientBase {
     HoodieTestUtils.createCompactionCommitFiles(fs, basePath, "001");
 
     HoodieTable table = HoodieTable.getHoodieTable(metaClient, config, jsc);
-    List<HoodieCleanStat> hoodieCleanStats = runCleaner(config);;
+    List<HoodieCleanStat> hoodieCleanStats = runCleaner(config);
     assertEquals("Must clean three files, one parquet and 2 log files", 3,
         getCleanStat(hoodieCleanStats, HoodieTestDataGenerator.DEFAULT_FIRST_PARTITION_PATH).getSuccessDeleteFiles()
             .size());
diff --git a/style/checkstyle.xml b/style/checkstyle.xml
index 51a291db9..649243b41 100644
--- a/style/checkstyle.xml
+++ b/style/checkstyle.xml
@@ -269,5 +269,7 @@
             <property name="illegalPkgs" value="org.apache.commons" />
             <property name="illegalClasses" value="java.util.Optional, com.google.common.base.Optional" />
         </module>
+
+        <module name="EmptyStatement" />
     </module>
 </module>