Skip to content

Commit

Permalink
sql: deflake TestUnsplit
Browse files Browse the repository at this point in the history
The bug here is that we were writing the new zone config after the GC job has
already deleted the old one.

Release note: None
  • Loading branch information
ajwerner committed Aug 15, 2022
1 parent fba19fc commit c4e6a61
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions pkg/sql/unsplit_range_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -309,11 +309,6 @@ func TestUnsplitRanges(t *testing.T) {
if _, err := sqlDB.Exec(tc.query); err != nil {
t.Fatal(err)
}
// Push a new zone config for a few tables with TTL=0 so the data
// is deleted immediately.
if _, err := sqltestutils.AddImmediateGCZoneConfig(sqlDB, tableDesc.GetID()); err != nil {
t.Fatal(err)
}

// Check GC worked!
testutils.SucceedsSoon(t, func() error {
Expand Down

0 comments on commit c4e6a61

Please sign in to comment.