[GitHub] [kafka] chia7712 commented on a change in pull request #10702: KAFKA-12940: Enable JDK 16 builds in Jenkins

[GitBox <gi...@apache.org>]

chia7712 commented on a change in pull request #10702:
URL: https://github.com/apache/kafka/pull/10702#discussion_r650504081



##########
File path: build.gradle
##########
@@ -353,6 +355,27 @@ subprojects {
     }
   }
 
+  // The suites are for running sets of tests in IDEs.
+  // Gradle will run each test class, so we exclude the suites to avoid redundantly running the tests twice.
+  def testsToExclude = ['**/*Suite.class']
+  // Exclude PowerMock tests when running with Java 16 until a version of PowerMock that supports Java 16 is released
+  // The relevant issues are https://github.com/powermock/powermock/issues/1094 and https://github.com/powermock/powermock/issues/1099
+  if (JavaVersion.current().isCompatibleWith(JavaVersion.VERSION_16)) {
+    testsToExclude.addAll([
+      // connect tests
+      "**/AbstractHerderTest.*", "**/ConnectClusterStateImplTest.*", "**/ConnectorPluginsResourceTest.*",
+      "**/ConnectorsResourceTest.*", "**/DistributedHerderTest.*", "**/FileOffsetBakingStoreTest.*",
+      "**/ErrorHandlingTaskTest.*", "**/KafkaConfigBackingStoreTest.*", "**/KafkaOffsetBackingStoreTest.*",
+      "**/KafkaBasedLogTest.*", "**/OffsetStorageWriterTest.*", "**/StandaloneHerderTest.*", 

Review comment:
       redundant whitespace

##########
File path: Jenkinsfile
##########
@@ -160,6 +161,25 @@ pipeline {
             echo 'Skipping Kafka Streams archetype test for Java 15'
           }
         }
+        

Review comment:
       redundant whitespace.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org