Skip to content

Commit 584cf20

Browse files
committed
kv: relax bounds for YCSB splits tests
We've seen variants of YCSB A and B fail with both higher and lower number of splits. In response, we've changed the bounds of both these variants independently. The commentary suggests the number of splits here should be the same for both variants; this patch makes it such, picking the max upper bound and min lower bound across the two variants. Closes #155951 Release note: None
1 parent 1656a4c commit 584cf20

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

pkg/cmd/roachtest/tests/split.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -335,9 +335,9 @@ func registerLoadSplits(r registry.Registry) {
335335
maxSize: 10 << 30, // 10 GB
336336
cpuThreshold: 100 * time.Millisecond, // 1/10th of a CPU per second.
337337
// YCSB/A has a zipfian distribution with 50% inserts and 50% updates.
338-
// The number of splits should be between 15-38 after 10 minutes with
338+
// The number of splits should be between 13-38 after 10 minutes with
339339
// 100ms threshold on 8vCPU machines.
340-
minimumRanges: 17,
340+
minimumRanges: 15,
341341
maximumRanges: 40,
342342
initialRangeCount: 2,
343343
load: ycsbSplitLoad{
@@ -362,7 +362,7 @@ func registerLoadSplits(r registry.Registry) {
362362
// The number of splits should be similar to YCSB/A.
363363
cpuThreshold: 100 * time.Millisecond, // 1/10th of a CPU per second.
364364
minimumRanges: 15,
365-
maximumRanges: 35,
365+
maximumRanges: 40,
366366
initialRangeCount: 2,
367367
load: ycsbSplitLoad{
368368
workload: "b",

0 commit comments

Comments
 (0)