From fc6f7c991394bc30892107bf5120a7cbeced6a40 Mon Sep 17 00:00:00 2001 From: Mark Sirek Date: Thu, 27 Apr 2023 20:01:22 +0000 Subject: [PATCH] logictestccl: add missing retries to multi_region_remote_access_error test PR #102103 added 2 tests to multi_region_remote_access_error which require retries in case zone configs did not propagate correctly. This adds the retries. Fixes #102406 Release note: None --- .../testdata/logic_test/multi_region_remote_access_error | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/pkg/ccl/logictestccl/testdata/logic_test/multi_region_remote_access_error b/pkg/ccl/logictestccl/testdata/logic_test/multi_region_remote_access_error index 47dfe3701c8d..c1c787ca4714 100644 --- a/pkg/ccl/logictestccl/testdata/logic_test/multi_region_remote_access_error +++ b/pkg/ccl/logictestccl/testdata/logic_test/multi_region_remote_access_error @@ -340,13 +340,14 @@ SELECT f_random_zip_code(); # A local lookup join with a CTE as input should succeed. -query IT +query IT retry WITH vtab AS (SELECT * FROM messages_rbt) SELECT vtab.account_id, vtab.message FROM vtab INNER LOOKUP JOIN messages_rbr rbr on vtab.account_id = rbr.account_id and rbr.crdb_region = 'ap-southeast-2' ---- # A CTE built from an INSERT which must access remote rows should fail. +retry statement error pq: Query has no home region\. Try adding a filter on parent\.crdb_region and/or on key column \(parent\.p_id\)\. For more information, see https://www.cockroachlabs.com/docs/stable/cost-based-optimizer.html#control-whether-queries-are-limited-to-a-single-region WITH vtab AS (INSERT INTO parent VALUES (6) RETURNING p_id) SELECT * FROM vtab