Skip to content

Commit

Permalink
Fix TestPartitionReader_ShouldNotMissRecordsIfFetchRequestContainPart…
Browse files Browse the repository at this point in the history
…ialFailuresWithConcurrentFetcherIsUsed flakyness (#9967)

Signed-off-by: Marco Pracucci <marco@pracucci.com>
  • Loading branch information
pracucci authored Nov 21, 2024
1 parent ef5cb17 commit 693efbc
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions pkg/storage/ingest/reader_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -2145,11 +2145,6 @@ func TestPartitionReader_ShouldNotMissRecordsIfFetchRequestContainPartialFailure
return nil, fmt.Errorf("expected 1 partition, got %d", len(req.Topics[0].Partitions)), true
}

// Simulate a 10% networking error rate.
if rand.Int()%10 == 0 {
return nil, errors.New("mocked error"), true
}

// Simulate a 10% Kafka error rate.
if rand.Int()%10 == 0 {
return &kmsg.FetchResponse{
Expand Down

0 comments on commit 693efbc

Please sign in to comment.