Skip to content

Commit 523a78d

Browse files
craig[bot]rafiss
andcommitted
Merge #152871
152871: rttanalysisccl: wait for replication before starting test r=spilchen a=rafiss This should help prevent a timeout that has occurred while setting the cluster setting and waiting for the new value to be present. fixes #152612 Release note: None Co-authored-by: Rafi Shamim <[email protected]>
2 parents 540d645 + 83abf47 commit 523a78d

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

pkg/ccl/benchccl/rttanalysisccl/multi_region_bench_test.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,9 @@ var reg = rttanalysis.NewRegistry(numNodes, rttanalysis.MakeClusterConstructor(f
2828
cluster, _, cleanup := multiregionccltestutils.TestingCreateMultiRegionCluster(
2929
tb, numNodes, knobs,
3030
)
31+
if err := cluster.WaitForFullReplication(); err != nil {
32+
tb.Fatal(err)
33+
}
3134
db := cluster.ServerConn(0)
3235
// Eventlog is async, and introduces jitter in the benchmark.
3336
if _, err := db.Exec("SET CLUSTER SETTING server.eventlog.enabled = false"); err != nil {

0 commit comments

Comments
 (0)