KAFKA-10787: Add import ordering checkstyle rule and configure an automatic formatter

README.md

@@ -227,11 +227,16 @@ There are two code quality analysis tools that we regularly run, spotbugs and ch
 Checkstyle enforces a consistent coding style in Kafka.
 You can run checkstyle using:
 
-    ./gradlew checkstyleMain checkstyleTest
+    ./gradlew checkstyleMain checkstyleTest spotlessCheck
 
 The checkstyle warnings will be found in `reports/checkstyle/reports/main.html` and `reports/checkstyle/reports/test.html` files in the
 subproject build directories. They are also printed to the console. The build will fail if Checkstyle fails.
 
+#### Spotless ####
+This plugin can review code by rules and can also help you check the code , it is disabled by default. Some of our code reviews use `spotless` instead of `checkstyle`:
+
+    ./gradlew spotlessApply
+
 #### Spotbugs ####
 Spotbugs uses static analysis to look for bugs in the code.
 You can run spotbugs using:

build.gradle

@@ -47,7 +47,8 @@ plugins {
   // Updating the shadow plugin version to 8.1.1 causes issue with signing and publishing the shadowed
   // artifacts - see https://github.com/johnrengelman/shadow/issues/901
   id 'com.github.johnrengelman.shadow' version '8.1.0' apply false
-  id 'com.diffplug.spotless' version '6.14.0' apply false // 6.14.1 and newer require Java 11 at compile time, so we can't upgrade until AK 4.0
+  //  6.14.0 and newer are need for java 11
+  id 'com.diffplug.spotless' version "6.13.0" apply false
 }
 
 ext {
@@ -198,6 +199,9 @@ def determineCommitId() {
   }
 }
 
+def spotlessApplyModules = ['']
+
+
 apply from: file('wrapper.gradle')
 
 if (repo != null) {
@@ -793,6 +797,16 @@ subprojects {
     skipProjects = [ ":jmh-benchmarks", ":trogdor" ]
     skipConfigurations = [ "zinc" ]
   }
+
+  if (project.name in spotlessApplyModules) {
+    apply plugin: 'com.diffplug.spotless'
+    spotless {
+      java {
+        importOrder('kafka', 'org.apache.kafka', 'com', 'net', 'org', 'java', 'javax', '', '\\#')
+        removeUnusedImports()
+      }
+    }
+  }
 }
 
 gradle.taskGraph.whenReady { taskGraph ->

checkstyle/checkstyle.xml

@@ -82,6 +82,10 @@
       <property name="file" value="${config_loc}/${importControlFile}"/>
     </module>
 
+    <!--   import order rule not config in the checkstyle.xml.
+        import order check by is checked by the spotless plugin.
+        More checks may be handed over to the spotless plugin in the future-->
+
     <!-- whitespace -->
     <module name="GenericWhitespace"/>
     <module name="NoWhitespaceBefore"/>