Skip to content

Commit de6fa3a

Browse files
committed
backfill: skip external tenant under deadlock in vector backfill test
`TestVectorIndexMergingDuringBackfillWithPrefix` is rather intensive, and we've seen a couple of failures due to general overload in the external-process mode under deadlock, so we'll just disable that mode under deadlock. Alternative idea I had was to exempt the tenant from rate limiting, but I wasn't able to reproduce the failure on gceworker, so I couldn't verify that it worked and decided to not go with it. Release note: None
1 parent 3b2811c commit de6fa3a

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

pkg/sql/backfill/backfill_test.go

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -346,6 +346,13 @@ func TestVectorIndexMergingDuringBackfillWithPrefix(t *testing.T) {
346346
blockBackfill := make(chan struct{})
347347
backfillBlocked := make(chan struct{})
348348

349+
var testTenant base.DefaultTestTenantOptions
350+
if syncutil.DeadlockEnabled {
351+
// The cluster can get overloaded under deadlock with external process
352+
// mode.
353+
testTenant = base.TestSkipForExternalProcessMode()
354+
}
355+
349356
ctx := context.Background()
350357
srv, db, _ := serverutils.StartServer(t, base.TestServerArgs{
351358
Knobs: base.TestingKnobs{
@@ -357,6 +364,7 @@ func TestVectorIndexMergingDuringBackfillWithPrefix(t *testing.T) {
357364
},
358365
},
359366
},
367+
DefaultTestTenant: testTenant,
360368
})
361369
defer srv.Stopper().Stop(ctx)
362370
sqlDB := sqlutils.MakeSQLRunner(db)

0 commit comments

Comments
 (0)