From fd6a1ac5f2760329711adc1e8a6a916127f4d105 Mon Sep 17 00:00:00 2001 From: Austen McClernon Date: Wed, 6 Sep 2023 14:35:11 -0400 Subject: [PATCH] ccl/multiregionccl: fix setting buglet in mr test Update the cluster setting statement to correctly surround the value duration in quotations.`5m` to `'5m'`. This was fixed in backports to release-23.1 (#110052) and release-22.2 (#110053). However, this was not caught on master due to `TestMultiRegionDataDriven` being skipped. Informs: #98020 Release note: None --- pkg/ccl/multiregionccl/datadriven_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/ccl/multiregionccl/datadriven_test.go b/pkg/ccl/multiregionccl/datadriven_test.go index c046b463a980..6ecfaa9b95f2 100644 --- a/pkg/ccl/multiregionccl/datadriven_test.go +++ b/pkg/ccl/multiregionccl/datadriven_test.go @@ -202,7 +202,7 @@ SET CLUSTER SETTING kv.closed_timestamp.side_transport_interval = '0.1s'; SET CLUSTER SETTING kv.closed_timestamp.propagation_slack = '0.5s'; SET CLUSTER SETTING kv.allocator.load_based_rebalancing = 'off'; SET CLUSTER SETTING kv.allocator.load_based_lease_rebalancing.enabled = false; -SET CLUSTER SETTING kv.allocator.min_lease_transfer_interval = 5m +SET CLUSTER SETTING kv.allocator.min_lease_transfer_interval = '5m' `, ";") { _, err = sqlConn.Exec(stmt)