Skip to content

Commit

Permalink
ccl/multiregionccl: fix setting buglet in mr test
Browse files Browse the repository at this point in the history
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 (cockroachdb#110052) and release-22.2 (cockroachdb#110053). However, this was not
caught on master due to `TestMultiRegionDataDriven` being skipped.

Informs: cockroachdb#98020
Release note: None
  • Loading branch information
kvoli committed Sep 6, 2023
1 parent 725501d commit fd6a1ac
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/ccl/multiregionccl/datadriven_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down

0 comments on commit fd6a1ac

Please sign in to comment.