Skip to content

Commit

Permalink
Merge #96160
Browse files Browse the repository at this point in the history
96160: logictest: retry support for "statement" and "query" commands r=msirek a=msirek

logictest: retry support for "statement" and "query" commands
----
This commit adds a retry command to logic tests which may be issued on a
separate line preceding either a "statement" or "query" command.     
It causes the statement to be retried with exponential backoff up 
to some maximum duration, e.g.
```
retry
statement error column "non_exist" does not exist
ALTER TABLE created_as_global SET LOCALITY REGIONAL BY ROW AS "non_exist"
```
This has the same effect as the retry option of the query command, but
now also supports "statement ok", "statement error" and "query error"
commands.

Retry of a query command may be specified by the standalone retry
command, the retry option of the query command, or both.

Fixes: #95668

Epic: CRDB-20535

Release note: None

logictest: vectorized off config for multi_region_remote_access_error test                            
----
This enables the `multiregion-9node-3region-3azs-vec-off` config for the
`multi_region_remote_access_error` test and adds retry commands prior to
statements.

Epic: CRDB-18645

Release note: None 

Co-authored-by: Mark Sirek <[email protected]>
  • Loading branch information
craig[bot] and Mark Sirek committed Feb 2, 2023
2 parents 22244a7 + e7ce7b9 commit c4257c9
Show file tree
Hide file tree
Showing 4 changed files with 104 additions and 133 deletions.
Loading

0 comments on commit c4257c9

Please sign in to comment.