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:
us...@infra.apache.org


Reply via email to