Skip to content

Commit

Permalink
MINOR: Reduce tries in RecordsIteratorTest to improve build time (apa…
Browse files Browse the repository at this point in the history
…che#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 <[email protected]>
  • Loading branch information
hachikuji authored Oct 31, 2022
1 parent fdd60cc commit c710ecd
Showing 1 changed file with 5 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ void testEmptyRecords(Records records) {
testIterator(Collections.emptyList(), records, true);
}

@Property
@Property(tries = 50)
public void testMemoryRecords(
@ForAll CompressionType compressionType,
@ForAll long seed
Expand All @@ -77,7 +77,7 @@ public void testMemoryRecords(
testIterator(batches, memRecords, true);
}

@Property
@Property(tries = 50)
public void testFileRecords(
@ForAll CompressionType compressionType,
@ForAll long seed
Expand All @@ -92,10 +92,10 @@ public void testFileRecords(
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);
Expand Down

0 comments on commit c710ecd

Please sign in to comment.