Skip to content

Commit

Permalink
sql: fix logic test for showing cluster setting defaults and origin
Browse files Browse the repository at this point in the history
A logic test that was verifying the results of
`ALTER TENANT ALL SET CLUSTER SETTING` was failing under stress runs as
the effects of this query can take some time to propagate to tenants. Adding
a retry to the select stmt fixes this issue by nesuring we wait for the
setting to change.

Fixes: #105485

Release note: None
  • Loading branch information
xinhaoz committed Jun 26, 2023
1 parent 15d43bb commit e6803eb
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/sql/logictest/testdata/logic_test/cluster_settings
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,7 @@ ALTER TENANT ALL SET CLUSTER SETTING sql.index_recommendation.drop_unused_durati
user root

onlyif config 3node-tenant-default-configs
query TTTT
query TTTT retry
SELECT variable, value, default_value, origin FROM [SHOW ALL CLUSTER SETTINGS]
WHERE variable IN ('sql.index_recommendation.drop_unused_duration')
----
Expand Down

0 comments on commit e6803eb

Please sign in to comment.