Skip to content

Commit

Permalink
logictest: retry relocate stmt in a couple of places
Browse files Browse the repository at this point in the history
The capabilities are propagated asynchronously, so previously we could
try to relocate ranges in the secondary tenant before the necessary
capability was picked up. This was recently changed in
5f2a4f8.

Release note: None
  • Loading branch information
yuzefovich committed Jul 24, 2024
1 parent 4af6b13 commit a81beba
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 4 deletions.
5 changes: 3 additions & 2 deletions pkg/sql/logictest/testdata/logic_test/distsql_tenant_locality
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,9 @@ statement ok
CREATE TABLE t (k INT PRIMARY KEY, v INT, FAMILY (k, v));
INSERT INTO t SELECT i, i FROM generate_series(1, 6) AS g(i)

# Upreplicate the table's range.
statement ok
# Upreplicate the table's range. We need a retry to guarantee that the
# capability has been picked up.
statement ok retry
ALTER TABLE t EXPERIMENTAL_RELOCATE VALUES (ARRAY[1, 2, 3], 0)

# Split the ranges in the table.
Expand Down
5 changes: 3 additions & 2 deletions pkg/sql/opt/exec/execbuilder/testdata/distsql_tenant_locality
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,9 @@ user root
statement ok
CREATE TABLE t (k INT PRIMARY KEY, v INT, FAMILY (k, v))

# Upreplicate the table's range.
statement ok
# Upreplicate the table's range. We need a retry to guarantee that the
# capability has been picked up.
statement ok retry
ALTER TABLE t EXPERIMENTAL_RELOCATE VALUES (ARRAY[1, 2, 3], 0)

# Split the ranges in the table.
Expand Down

0 comments on commit a81beba

Please sign in to comment.