Skip to content

Commit

Permalink
sql: deflake TestBackfillWithProtectedTS
Browse files Browse the repository at this point in the history
We have seen that the DELETE and INSERT step can fail in this test,
which occurs before we manually send the table into the GC queue. To
reduce the chance of this happening, we use a longer gc.ttl.

Release note: None
  • Loading branch information
rafiss committed Jan 24, 2025
1 parent 8372591 commit e1d9098
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pkg/sql/backfill_protected_timestamp_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -397,10 +397,10 @@ func TestBackfillWithProtectedTS(t *testing.T) {
t.Run(tc.name, func(t *testing.T) {
for _, sql := range []string{
"SET CLUSTER SETTING sql.stats.automatic_collection.enabled = false",
"ALTER DATABASE defaultdb CONFIGURE ZONE USING gc.ttlseconds = 1",
"ALTER DATABASE defaultdb CONFIGURE ZONE USING gc.ttlseconds = 5",
"DROP TABLE IF EXISTS t CASCADE",
"CREATE TABLE t(n int)",
"ALTER TABLE t CONFIGURE ZONE USING range_min_bytes = 0, range_max_bytes = 67108864, gc.ttlseconds = 1",
"ALTER TABLE t CONFIGURE ZONE USING range_min_bytes = 0, range_max_bytes = 67108864, gc.ttlseconds = 5",
fmt.Sprintf("INSERT INTO t(n) SELECT * FROM generate_series(1, %d)", initialRowCount),
} {
r.Exec(t, sql)
Expand Down

0 comments on commit e1d9098

Please sign in to comment.