This is an automated email from the ASF dual-hosted git repository.

jgus pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/kafka.git


The following commit(s) were added to refs/heads/trunk by this push:
     new c710ecd071f MINOR: Reduce tries in RecordsIteratorTest to improve 
build time (#12798)
c710ecd071f is described below

commit c710ecd071fef2796a3a626a0464933605358474
Author: Jason Gustafson <ja...@confluent.io>
AuthorDate: Mon Oct 31 09:29:19 2022 -0700

    MINOR: Reduce tries in RecordsIteratorTest to improve build time (#12798)
    
    `RecordsIteratorTest` takes the longest times in recent builds (even 
including integration tests). The default of 1000 tries from jqwik is probably 
overkill and causes the test to take 10 minutes locally. Decreasing to 50 tries 
reduces that to less than 30s.
    
    Reviewers: David Jacot <dja...@confluent.io>
---
 .../org/apache/kafka/raft/internals/RecordsIteratorTest.java   | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git 
a/raft/src/test/java/org/apache/kafka/raft/internals/RecordsIteratorTest.java 
b/raft/src/test/java/org/apache/kafka/raft/internals/RecordsIteratorTest.java
index 9dfbfd62fbf..9676831b30e 100644
--- 
a/raft/src/test/java/org/apache/kafka/raft/internals/RecordsIteratorTest.java
+++ 
b/raft/src/test/java/org/apache/kafka/raft/internals/RecordsIteratorTest.java
@@ -66,7 +66,7 @@ public final class RecordsIteratorTest {
         testIterator(Collections.emptyList(), records, true);
     }
 
-    @Property
+    @Property(tries = 50)
     public void testMemoryRecords(
         @ForAll CompressionType compressionType,
         @ForAll long seed
@@ -77,7 +77,7 @@ public final class RecordsIteratorTest {
         testIterator(batches, memRecords, true);
     }
 
-    @Property
+    @Property(tries = 50)
     public void testFileRecords(
         @ForAll CompressionType compressionType,
         @ForAll long seed
@@ -92,10 +92,10 @@ public final class RecordsIteratorTest {
         fileRecords.close();
     }
 
-    @Property
+    @Property(tries = 50)
     public void testCrcValidation(
-            @ForAll CompressionType compressionType,
-            @ForAll long seed
+        @ForAll CompressionType compressionType,
+        @ForAll long seed
     ) throws IOException {
         List<TestBatch<String>> batches = createBatches(seed);
         MemoryRecords memRecords = buildRecords(compressionType, batches);

Reply via email to